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

Reply via email to