Re: [HACKERS] backend/utils/adt/float.c uses non-existent NAN value - Mailing list pgsql-hackers

From jwieck@debis.com (Jan Wieck)
Subject Re: [HACKERS] backend/utils/adt/float.c uses non-existent NAN value
Date
Msg-id m102AAN-000EBQC@orion.SAPserv.Hamburg.dsh.de
Whole thread Raw
In response to backend/utils/adt/float.c uses non-existent NAN value  (Taral <taral@cyberjunkie.com>)
Responses Re: [HACKERS] backend/utils/adt/float.c uses non-existent NAN value  (Bruce Momjian <maillist@candle.pha.pa.us>)
List pgsql-hackers
>
> Seems at some point someone decided not to be machine-independent in
> backend/utils/adt/float.c
>
> Redhat 5.2 system with gcc 2.8.1 doesn't define NAN unless _GNU_SOURCE is
> defined first. Although include/utils/dt.h *does* check for NAN (and HUGE_VAL)
> before using them, the float.c doesn't.
>
> What happened?

    Don't know exactly - but

    the new NUMERIC data type started recently a discussion about
    NAN, because I tried  to  be  able  to  convert  NAN  from/to
    float's, and that started some trouble. I'm not sure how this
    went into adt/float.c. At least I think we should fix all the
    NAN handling for v6.5.


Jan

--

#======================================================================#
# It's easier to get forgiveness for being wrong than for being right. #
# Let's break this rule - forgive me.                                  #
#======================================== jwieck@debis.com (Jan Wieck) #

pgsql-hackers by date:

Previous
From: Vadim Mikheev
Date:
Subject: Re: [HACKERS] I need a PostgreSQL vacation
Next
From: Magnus Hagander
Date:
Subject: RE: [PATCHES] Another libpq-be patch