Re: NaN/Inf fix for ECPG

From: Boszormenyi Zoltan <zb(at)cybertec(dot)at>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Meskes <meskes(at)postgresql(dot)org>
Cc: Rémi Zara <remi_zara(at)mac(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: NaN/Inf fix for ECPG
Date: 2010-02-26 11:12:10
Message-ID: 4B87AC8A.4020003@cybertec.at
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane írta:
> Boszormenyi Zoltan <zb(at)cybertec(dot)at> writes:
>
>> Can you try whether the first patch (missing float.h from data.c)
>> solves the problem? And together with the 2nd one? In that
>> patch I fixed the order of float.h and math.h in nan_test.pgc,
>> which is the opposite of the order found in e.g. backend/utils/adt/float.c.
>>
>
>
>> The 3rd patch is explicit about NetBSD/mips but it doesn't feel right.
>>
>
> The third patch is surely wrong. We don't need to do that in the
> backend's instance of get_float8_nan, so ecpglib shouldn't need it
> either.
>
> I suspect that the ultimate cause of this is either one of the header
> inclusion inconsistencies you found, or something associated with
> not pulling in all the stuff that postgres.h does. port/netbsd.h
> is empty though, so it's not immediately clear what might be missing.
>

ecpglib/data.c includes "postgres_fe.h", so it should be pulling
everything that's relevant from port/*, right?

Michael, can we try to install the first two patches?
They wouldn't hurt.

Thanks in advance,
Zoltán Böszörményi

--
Bible has answers for everything. Proof:
"But let your communication be, Yea, yea; Nay, nay: for whatsoever is more
than these cometh of evil." (Matthew 5:37) - basics of digital technology.
"May your kingdom come" - superficial description of plate tectonics

----------------------------------
Zoltán Böszörményi
Cybertec Schönig & Schönig GmbH
http://www.postgresql.at/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Stark 2010-02-26 12:10:07 Re: Hot Standby query cancellation and Streaming Replication integration
Previous Message Jeroen Vermeulen 2010-02-26 10:20:17 Re: Avoiding bad prepared-statement plans.