Mark Dilger <hornschnor...@gmail.com> writes: > You appear to be using a #define macro to wrap a function of the same name > with the code:
> #define copyObject(obj) ((typeof(obj)) copyObject(obj)) > I don't necessarily have a problem with that, but it struck me as a bit odd, > and > grep'ing through the sources, I don't see anywhere else in the project where > that > is done for a function of the same number of arguments, and only one other > place where it is done at all: I agree that that seems like a bad idea. Better to rename the underlying function --- for one thing, that provides an "out" if some caller doesn't want this behavior. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers