Re: A minor correction in comment in heaptuple.c

On Tue, 18 Jun 2013 19:19:40 +0200
Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
> 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.

Actually, the comment is not for the function. It only describes the
two lines that follow. In C the string "NULL" is commonly a reference
to C's NULL value. Anyone reading C code can be excused for assuming
that if it isn't otherwise clear. How about "Indicate that the
attribute is NULL if out of range..."?

Although, the more I think about it, the more I think that the comment
is both confusing and superfluous. The code itself is much clearer.

--
D'Arcy J.M. Cain <darcy(at)druid(dot)net> | Democracy is three wolves
http://www.druid.net/darcy/ | and a sheep voting on
+1 416 788 2246 (DoD#0082) (eNTP) | what's for dinner.
IM: darcy(at)Vex(dot)Net, VOIP: sip:darcy(at)Vex(dot)Net