Hi, I was about to add a few more uses of castNode, which made me think.
You proposed replacing: On 2016-12-31 12:08:22 -0500, Peter Eisentraut wrote: > There is a common coding pattern that goes like this: > > RestrictInfo *rinfo = (RestrictInfo *) lfirst(lc); > Assert(IsA(rinfo, RestrictInfo)); with > +#define castNode(_type_,nodeptr) (AssertMacro(!nodeptr || > IsA(nodeptr,_type_)), (_type_ *)(nodeptr)) (now an inline function, but that's besides my point) Those aren't actually equivalent, because of the !nodeptr. IsA() crashes for NULL pointers, but the new code won't. Which means 9ba8a9ce4548b et al actually weakened some asserts. Should we perhaps have one NULL accepting version (castNodeNull?) and one that separately asserts that ptr != NULL? Greetings, Andres Freund -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
