David Rowley <david.rowley@2ndquadrant.com> writes:
> Updated patch attached again. This time due to a newly added call to
> LookupFuncName() in 1fb57af92.
Hmm ... I'd not looked at this before, but now that I do, the new API
for LookupFuncName seems mighty confused, or at least confusingly
documented. It's not clear what the combinations of the flags actually
do, or why you'd want to use them.
I wonder whether you'd be better off replacing the two bools with an
enum, or something like that.
regards, tom lane