On Sun, Feb 14, 2010 at 12:24 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: > 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.
You are correct. I realized that this morning while I was shaving. Sorry about that. > 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. Good idea. > I didn't check all the individual calls, but it looks generally > sane except for those points. Will fix and commit. ...Robert -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers