Re: _isnan() on Windows - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: _isnan() on Windows
Date
Msg-id 20180710175128.ubudbinw6qr5bwen@alvherre.pgsql
Whole thread Raw
In response to _isnan() on Windows  (Emre Hasegeli <emre@hasegeli.com>)
Responses Re: _isnan() on Windows
List pgsql-hackers
On 2018-Jul-10, Emre Hasegeli wrote:

> isnan() function is evidently not present on <math.h> on Windows
> before Visual Studio 2013.  We define it on win32_port.h using
> _isnan().  However _isnan() is also not present.  It is on <float.h>.
> The patch is attached to include this from win32_port.h.
> 
> Thanks to Thomas Munro for point this out to me [1].  It is hard to
> notice this issue without testing the changes on Windows.

Oh, it looks like commits 33a7101281c6, 8e211f539146, 86dbbf20d849
(probably others) papered over this by the expedient of adding #include
<float.h> to random .c files rather than your patch, which seems the
proper fix.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Emre Hasegeli
Date:
Subject: Re: [PATCH] Improve geometric types
Next
From: Alvaro Herrera
Date:
Subject: Re: no partition pruning when partitioning using array type