Re: ECPGset_var - Mailing list pgsql-hackers

From Boszormenyi Zoltan
Subject Re: ECPGset_var
Date
Msg-id 4B5F30C5.5080501@cybertec.at
Whole thread Raw
In response to Re: ECPGset_var  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: ECPGset_var
Re: ECPGset_var
List pgsql-hackers
Tom Lane írta:
> Boszormenyi Zoltan <zb@cybertec.at> writes:
>
>> Michael Meskes Ă­rta:
>>
>>> Zoltan, could you please look into this:
>>> http://pgbuildfarm.org/cgi-bin/show_log.pl?nm=dugong&dt=2010-01-26%2011:05:01 ?
>>> Apparently dugong creates the ECPGset_var statements in a different order than
>>> the other archs.
>>>
>
>
>> It seems mongoose also fails the same way,
>> the common factor seems to be ICC, other machines
>> with various GCC versions are green . I'll look into it.
>>
>
> The problem is you've got calls like this:
>
>                 $$ = cat_str(4,
>                              adjust_outofscope_cursor_vars(this, true),
>                              adjust_outofscope_cursor_vars(this, false),
>                              make_str("ECPG_informix_reset_sqlca();"),
>                              comment);
>
> in which the order of evaluation of cat_str's parameters is unspecified,
> but adjust_outofscope_cursor_vars has got order-dependent side effects.
>
>             regards, tom lane
>
>

Thanks for analyzing it, patch is attached. I downloaded
and installed ICC, so I could reproduce the difference.

Also, another "bug" is fixed in one regression test,
it seems NaN is different across platforms, so
we must not test for it either.

Best regards,
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/


Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Review: listagg aggregate
Next
From: "David E. Wheeler"
Date:
Subject: Re: Review: listagg aggregate