Re: [PATCH] Improve geometric types

On 06/05/2018 06:32 PM, Emre Hasegeli wrote:
>> Those underscore-prefixed names are defined in Microsoft's
>> <float.h>[3][4]. So now I'm wondering if win32_port.h needs to
>> #include <float.h> if (_MSC_VER < 1800).
>
> I don't have the C experience to decide the correct way. There are
> currently many .c files that are including float.h conditionally or
> unconditionally. The condition they use is "#ifdef _MSC_VER" without
> a version.
>
> One idea is to include float.h from the new utils/float.h file
> together with math.h, and remove those includes from the .c files
> which would include utils/float.h. We can do this only, or together
> with what you suggest, or by also keeping the includes on the .c
> files. Which way do you think is the proper?
>

Do we have any solution to the float.h include issues on Windows? I
don't have any Windows box at hand so I can't verify it, but just using
"#ifdef _MSC_VER" seems OK to me (and it's used elsewhere). Thomas, why
do you think the version number restriction is needed here? I don't see
the version mentioned in the MS docs you linked either.

Once this gets resolved, I'd like to get this committed ... so if you
have other objections, please speak now.

regards

--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services