Thread: Re: [HACKERS] column labels now with obligatory 'as'

Re: [HACKERS] column labels now with obligatory 'as'

From
darrenk@insightdist.com (Darren King)
Date:
> Have just tried 9. Jan snapshot on AIX 4.1.5,
> compiles with gcc not with cc (fails in heaptuple.c), aix won't get
> defined though, did a -Daix

IBM compiler doesn't like the (void)NULLs in the heap_getattr and
StrNCpy macros.  Change to (bool)NULL in heap_getattr in include/
access/heapam.h and to (char)NULL in StrNCpy in include/c.h. In 6.2,
heap_getattr has (char)NULL (which is what bool is, so this worked),
but gcc gave warnings, so they were changed to _errors_ to "fix" them
for 6.3.

darrenk

Re: [HACKERS] column labels now with obligatory 'as'

From
"Thomas G. Lockhart"
Date:
> > Have just tried 9. Jan snapshot on AIX 4.1.5,
> > compiles with gcc not with cc (fails in heaptuple.c), aix won't get
> > defined though, did a -Daix
>
> IBM compiler doesn't like the (void)NULLs in the heap_getattr and
> StrNCpy macros.  Change to (bool)NULL in heap_getattr in include/
> access/heapam.h and to (char)NULL in StrNCpy in include/c.h. In 6.2,
> heap_getattr has (char)NULL (which is what bool is, so this worked),
> but gcc gave warnings, so they were changed to _errors_ to "fix" them
> for 6.3.

Ack! It looks to me like the StrNCpy macro needs (void *)NULL or just
NULL, not (void)NULL. Same with heap_getattr. Was this a workaround for
a compiler problem on another platform, or are we mixing type coersions?

                                                         - Tom


Re: [HACKERS] column labels now with obligatory 'as'

From
Bruce Momjian
Date:
>
> > > Have just tried 9. Jan snapshot on AIX 4.1.5,
> > > compiles with gcc not with cc (fails in heaptuple.c), aix won't get
> > > defined though, did a -Daix
> >
> > IBM compiler doesn't like the (void)NULLs in the heap_getattr and
> > StrNCpy macros.  Change to (bool)NULL in heap_getattr in include/
> > access/heapam.h and to (char)NULL in StrNCpy in include/c.h. In 6.2,
> > heap_getattr has (char)NULL (which is what bool is, so this worked),
> > but gcc gave warnings, so they were changed to _errors_ to "fix" them
> > for 6.3.
>
> Ack! It looks to me like the StrNCpy macro needs (void *)NULL or just
> NULL, not (void)NULL. Same with heap_getattr. Was this a workaround for
> a compiler problem on another platform, or are we mixing type coersions?

Prevented the warning 'right-hand side of conditional skipped...'  Try
and and see what you get.  It has to do with macro usage and constant
parameters.


--
Bruce Momjian
maillist@candle.pha.pa.us