From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | t-ishii(at)sra(dot)co(dot)jp |
Cc: | Taral <taral(at)taral(dot)net>, hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] inet data type regression test fails |
Date: | 1999-05-11 01:05:08 |
Message-ID: | 199905110105.VAA26651@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> > > > > int addr = htonl(ntohl(ip_v4addr(ip)) | (0xffffffff >> ip_bits(ip)));
> > >
> > > There needs to be a UL on the end of that constant. Otherwise it depends
> > > on whether or not the compiler chooses to make it signed or unsigned. Not
> > > only that, but shifting by >=32 is undefined... Intel chipsets will go mod
> > > 32 and change 32 to 0.
> > >
> >
> > Anyone want to supply a patch?
>
> This has been already fixed. Now it looks like:
>
> unsigned long mask = 0xffffffff;
>
> if (ip_bits(ip) < 32)
> mask >>= ip_bits(ip);
> addr = htonl(ntohl(ip_v4addr(ip)) | mask);
Oh. Very nice.
--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Taral | 1999-05-11 01:05:28 | Re: [HACKERS] inet data type regression test fails |
Previous Message | Taral | 1999-05-11 00:58:58 | Optimization info |