didier <did...@gmail.com> writes:
> Trivial patch:
> - remove a gcc warning (since commit 7a0574b5)
> expression which evaluates to zero treated as a null pointer constant of
> type 'HeapTuple' (aka 'struct HeapTupleData *')
Hmm, the initializations "HeapTuple newtuple = false" are certainly
bogus-looking and not per project style; I wonder who's to blame for
those? (I do not see what 7a0574b5 would have had to do with it;
that didn't affect any backend code.)
> - always use "if (newtuple == NULL)" rather than mixing !newtuple and
> newtuple == NULL
Don't particularly agree with these changes though. "if (!ptr)" is
a very common C idiom, and no programmer would tolerate a compiler
that warned about it.
regards, tom lane
