I wanted to test the new patches on AIX 5L but the cvs server seem to
reject me (it worked till yesterday). Sigh.
--
Tatsuo Ishii
> OK, I backed out your previous patch and applied this one.
>
> Thanks.
>
> ---------------------------------------------------------------------------
>
> >
> > > "Zeugswetter Andreas SB SD" <ZeugswetterA@spardat.at> writes:
> > >
> > > /* Plain "long int" fits, use it */
> > > + #if SIZEOF_INT8 == 0
> > > typedef long int int64;
> > > + #endif
> > > + #if SIZEOF_UINT8 == 0
> > > typedef unsigned long int uint64;
> > > + #endif
> > >
> > >
> > > This coding appears to assume "if the platform defines int8, then
> > > it will define int64 as well". Seems mighty fragile to me.
> >
> > Well the absolute correct solution would involve all of:
> > int8, int16, int32, int64 and separately uint8, uint16, uint32, uint64
> >
> > The previous patch grouped:
> > int8, int16 and int32
> > uint8, uint16 and uint32
> > int64 and uint64 <-- this grouping is wrong on AIX 4.3.3 and below
> >
> > If you prefer to make 4 groups out of this you could apply this patch.
> >
> > Andreas
>
> Content-Description: int8-newpatch2
>
> [ Attachment, skipping... ]
>
> --
> Bruce Momjian | http://candle.pha.pa.us
> pgman@candle.pha.pa.us | (610) 853-3000
> + If your life is a hard drive, | 830 Blythe Avenue
> + Christ can be your backup. | Drexel Hill, Pennsylvania 19026
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
>
