Bruce Momjian <bruce(at)momjian(dot)us> writes:
> I have made some more progress on this patch.
I'm not convinced that you're fixing things so much as doing your best
to destroy IEEE-compliant float arithmetic behavior.
I think what we should probably consider is removing CheckFloat4Val
and CheckFloat8Val altogether, and just letting the float arithmetic
have its head. Most modern hardware gets float arithmetic right per
spec, and we shouldn't be second-guessing it.
A slightly less radical proposal is to reject only the case where
isinf(result) and neither input isinf(); and perhaps likewise with
respect to NaNs.
regards, tom lane
In response to
pgsql-hackers by date
|Next:||From: Simon Riggs||Date: 2006-12-27 19:24:46|
|Subject: Re: Bitmap index thoughts|
|Previous:||From: dakotali kasap||Date: 2006-12-27 19:08:24|
|Subject: Re: how to add my source file?|
pgsql-patches by date
|Next:||From: Alvaro Herrera||Date: 2006-12-27 20:22:35|
|Subject: Re: Allow the identifier length to be increased via a configure option|
|Previous:||From: Bruce Momjian||Date: 2006-12-27 18:44:31|
|Subject: Re: [BUGS] BUG #2846: inconsistent and confusing handling of|