Robert Haas <robertmh...@gmail.com> writes: > OK. In that case, any objections to my applying the attached patch, > which I believe implements this as you suggested?
Um, did you test this version? It looks like the macros are still defined according to the idea that SearchSysCache takes five arguments. Also, I'd suggest adding explicit comments to syscache.h suggesting that SearchSysCache etc are meant to be called via the macros rather than directly. I didn't check all the individual calls, but it looks generally sane except for those points. 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