Re: HeapTupleData.t_self garbage values - Mailing list pgsql-hackers

From Tom Lane
Subject Re: HeapTupleData.t_self garbage values
Date
Msg-id 28440.1268341959@sss.pgh.pa.us
Whole thread Raw
In response to Re: HeapTupleData.t_self garbage values  ("Kevin Grittner" <Kevin.Grittner@wicourts.gov>)
Responses Re: HeapTupleData.t_self garbage values
List pgsql-hackers
"Kevin Grittner" <Kevin.Grittner@wicourts.gov> writes:
> At a minimum, it might be good to qualify the comment in htup.h and
> add a comment where there is an exception.  This can be startling in
> a debugger if you don't know that the comment isn't really true. 
> (And I've found another place where t_tableOid isn't set, but it is
> apparently benign; that's without an exhaustive search.)
> I could put forward a comment-only patch per the above if there are
> no objections.

I don't want to put weasel wording into the comment.  If you're bothered
by the discrepancy then we should fix the code to initialize all the
struct fields.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [patch] build issues on Win32
Next
From: Bruce Momjian
Date:
Subject: Re: Warning about invalid .pgpass passwords