Hi,
On 2013-06-18 17:56:34 +0900, Amit Langote wrote:
> Should it be: "return true if attnum is out of range according to the
> tupdesc" instead of "return NULL if attnum is out of range according
> to the tupdesc" at src/backend/access/common/heaptuple.c: 1345
>
> /*
> * return true if attnum is out of range according to the tupdesc
> */
> if (attnum > tupleDesc->natts)
> return true;
Well, true actually tells us that the attribute is null, since that's
the purpose of the function:
/** slot_attisnull* Detect whether an attribute of the slot is null, without* actually fetching it.*/
I think the comment is more meaningfull before the change since it tells
us how nonexisting columns are interpreted.
Greetings,
Andres Freund
-- Andres Freund http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training &
Services