On Mon, Apr 24, 2006 at 01:16:04PM -0500, Kris Jurka wrote: > > > On Mon, 24 Apr 2006, Gevik Babakhani wrote: > > >I noticed the following compile warnings. Perhaps someone is interested > >to know about them. > > Also I was testing a gcc 4.2 snapshot (20060419) and it has a whole lot of > warnings stemming from heap_getattr's isnull check: > > aclchk.c:791: warning: the address of 'isNull', will always evaluate as > 'true'
Perhaps someone could check if changing the test explicitly check against NULL: > ((attnum) > (int) (tup)->t_data->t_natts) ? \ > ( \ > (((isnull) != NULL)? (*(isnull) = true) : (dummyret)NULL), \ > (Datum)NULL \ removes the warning. It seems silly for the GCC people to add warnings for this kind of stuff without a simple way to bypass it... -- Martijn van Oosterhout <kleptog@svana.org> http://svana.org/kleptog/ > From each according to his ability. To each according to his ability to > litigate.
signature.asc
Description: Digital signature
