Re: Incompatible trig error handling - Mailing list pgsql-hackers

From Noah Misch
Subject Re: Incompatible trig error handling
Date
Msg-id 20150613050900.GA289675@tornado.leadboat.com
Whole thread Raw
In response to Re: Incompatible trig error handling  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Wed, Apr 29, 2015 at 05:11:48PM -0700, Tom Lane wrote:
> John Gorman <johngorman2@gmail.com> writes:
> > Two of the trigonometry functions have differing error condition behavior
> > between Linux and OSX. The Linux behavior follows the standard set by the
> > other trig functions.
> 
> We have never considered it part of Postgres' charter to try to hide
> platform-specific variations in floating-point behavior.  If we did,
> we'd spend all our time doing that rather than more productive stuff.
> 
> In particular, it appears to me that both of these behaviors are allowed
> per the POSIX standard, which makes it very questionable why we should
> insist that one is correct and the other is not.
> 
> In addition, the proposed patch turns *all* cases that return NaN into
> errors, which is wrong at least for the case where the input is NaN.

OS X is a MATH_ERREXCEPT, !MATH_ERRNO platform.  PostgreSQL wrongly assumes
that all platforms are MATH_ERRNO platforms.  The correct fix is to use
fetestexcept() on !MATH_ERRNO platforms.



pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: Time to fully remove heap_formtuple() and friends?
Next
From: Michael Meskes
Date:
Subject: Re: Collection of memory leaks for ECPG driver