On Wed, Jan 6, 2010 at 1:38 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: > Robert Haas <robertmh...@gmail.com> writes: >> Well, that comment is a bit misleading too, since a pointer with a >> NULL value might work but a literal NULL certainly doesn't. > > I think "(bool *) NULL" would work. What your compiler is complaining > about is trying to dereference a "void *" expression. > > In practice, the people we'd need to reach with a comment would be ones > who had working code before --- and working code, in this context, would > most likely be code that was passing a pointer variable that contained > null. But as I said, I don't think it really requires any comment.
I was less thinking of people whose code might break and more thinking of people who might be trying to understand the preconditions for using the macro. But on further reflection I think a lot more documentation would be needed to really make that clear, so I'll skip it for now. ...Robert -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers