Robert Haas <robertmhaas@gmail.com> writes:
> Spoke with Bruce on IM and we think the best option is to just remove
> the NULL tests. Since it's been this way for 11 years, presumably
> nobody is trying to use it with a NULL fourth argument.
> Proposed patch attached.
There are a number of is-null checks in related code that ought to go
away too --- look at heap_getattr, nocachegetattr, etc. Our principle
here ought to be that none of the field-fetching routines allow a null
pointer.
I wouldn't bother with those added comments. They wouldn't have been
there if the code had always been like this. If you feel a need to
have a comment, it should be more like "Before Postgres 8.5, the isnull
argument could be a null pointer, but we no longer allow that". That
way tells people that there was a change here that might affect their
code, whereas the addition you suggest wouldn't flag that.
regards, tom lane