Re: A minor correction in comment in heaptuple.c - Mailing list pgsql-hackers

From Andres Freund
Subject Re: A minor correction in comment in heaptuple.c
Date
Msg-id 20130618171940.GH5646@awork2.anarazel.de
Whole thread Raw
In response to Re: A minor correction in comment in heaptuple.c  ("D'Arcy J.M. Cain" <darcy@druid.net>)
Responses Re: A minor correction in comment in heaptuple.c
List pgsql-hackers
On 2013-06-18 13:14:30 -0400, D'Arcy J.M. Cain wrote:
> On Tue, 18 Jun 2013 11:38:45 +0200
> Andres Freund <andres@2ndquadrant.com> wrote:
> > > How about "check if attnum is out of range according to the tupdesc"
> > > instead?
> > 
> > I can't follow. Minus the word 'NULL' - which carries meaning - your
> > suggested comment pretty much is the same as the existing comment
> > except that you use 'check' instead of 'return'.
> 
> The difference is that I say what the purpose of the function is but
> don't say what it actually returns.  The code itself does that.
>
> > Original:
> >     /*
> >      * return NULL if attnum is out of range according to the
> > tupdesc */
> 
> Obviously wrong so it should be changed.

The NULL refers to the *meaning* of the function (remember, it's called
slot_attisnull) . Which is to test whether an attribute is null. Not to
a C NULL.

Greetings,

Andres Freund

-- Andres Freund                       http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training &
Services



pgsql-hackers by date:

Previous
From: Jeremy Evans
Date:
Subject: LEFT JOIN LATERAL can remove rows from LHS
Next
From: Josh Berkus
Date:
Subject: Re: ASYNC Privileges proposal