From: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Suppress compiler warnings in ecpg test on newer Windows toolcha |
Date: | 2017-03-08 02:47:26 |
Message-ID: | CAKJS1f-BHceaFzZScFapDV48gUVM2CAOBfhkgffdqXzFb+kwew@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 25 February 2017 at 10:45, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Suppress compiler warnings in ecpg test on newer Windows toolchains.
>
> nan_test.pgc supposed that it could unconditionally #define isnan()
> and isinf() on WIN32. This was evidently copied at some point from
> src/include/port/win32.h, but nowadays there's a test on _MSC_VER
> there. Make nan_test.pgc look the same.
>
> Per buildfarm warnings. There's no evidence this produces anything
> worse than a warning, and besides it's only a test case, so I don't
> feel a need to back-patch.
>
> Branch
> ------
> master
>
> Details
> -------
> http://git.postgresql.org/pg/commitdiff/c5658a0764d5ac5ea8c2c11d27c62d5472234227
>
> Modified Files
> --------------
> .../ecpg/test/expected/pgtypeslib-nan_test.c | 102 +++---
> .../ecpg/test/expected/pgtypeslib-nan_test.stderr | 354 ++++++++++-----------
> src/interfaces/ecpg/test/pgtypeslib/nan_test.pgc | 2 +
> 3 files changed, 231 insertions(+), 227 deletions(-)
This seems to have caused some new compiler warnings [1] on earlier
MSVC toolchains.
The reason seems to be that these older versions require float.h to be
included for _isnan() [2]
So anyway, we seem to be including float.h in all the other places
we're using isnan(), so the attached adds float.h to the files where
we're getting the warnings too.
[1] https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=thrips&dt=2017-03-07%2022%3A37%3A28&stg=make
[2] https://msdn.microsoft.com/en-us/library/tzthab44(v=vs.110).aspx
--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
Attachment | Content-Type | Size |
---|---|---|
isnan_msvc_warning_fixes.patch | application/octet-stream | 1.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2017-03-08 03:46:16 | pgsql: Fix segfault in ALTER PUBLICATION/SUBSCRIPTION RENAME |
Previous Message | Robert Haas | 2017-03-07 22:08:24 | pgsql: hash: Refactor hash index creation. |