Re: SunOS patch for memcmp() - Mailing list pgsql-patches

From Tom Lane
Subject Re: SunOS patch for memcmp()
Date
Msg-id 5649.1008818958@sss.pgh.pa.us
Whole thread Raw
In response to Re: SunOS patch for memcmp()  (Tatsuo Ishii <t-ishii@sra.co.jp>)
Responses Re: SunOS patch for memcmp()
Re: SunOS patch for memcmp()
List pgsql-patches
Tatsuo Ishii <t-ishii@sra.co.jp> writes:
> Ok. I have tested patches from Bruce. Now tests for bit passed.
> Remaining issues seem that strtol() is broken on SunOS4, not detecting
> an overflow, which causes int4 and some of other tests failure.

Looks like you might also want to use horology-no-DST-before-1970.out.

> Should we use our own strtol()?

Mumble.  The memcmp fix was all from well-tested parts, but I don't
think we have a canned test for strtol breakage available.  Also,
I believe that the SunOS strtol lossage has been known and tolerated
in previous Postgres releases.  Since it's not a regression, I'm going
to change sides and vote with Peter: let's not fix this one now.

            regards, tom lane

pgsql-patches by date:

Previous
From: "Christopher Kings-Lynne"
Date:
Subject: Geometry patch for freebsd alpha
Next
From: Tatsuo Ishii
Date:
Subject: Re: SunOS patch for memcmp()