From: | Boszormenyi Zoltan <zb(at)cybertec(dot)at> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Michael Meskes <meskes(at)postgresql(dot)org>, PostgreSQL Hacker <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: ECPG check variables hidden by locals v2 |
Date: | 2010-04-13 04:39:35 |
Message-ID: | 4BC3F587.9040204@cybertec.at |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Yes, he applied the first version without seeing this one,
then he asked for a re-diff privately.
Bruce Momjian írta:
> FYI, I think Michael Meskes applied this patch, though I didn't see you
> emailed that it was applied.
>
> ---------------------------------------------------------------------------
>
> Boszormenyi Zoltan wrote:
>
>> Hi,
>>
>> here's a little beautified patch:
>> - more logical parameter order in ECPGdump_a_type()
>> - use mm_strdup() instead of strdup() (I notoriously forget this)
>> - actually bail out with ET_FATAL if the local variable is
>> of a different type than the global variable that was used in
>> the DECLARE in the global scope
>>
>> Although with this patch we can only detect variables under
>> DECLARE SECTIONs, so we can't detect the scenario
>> in the attached test case. Should we? This test code would be
>> a good candidate for the Underhanded C Contest. :-)
>>
>> 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, skipping... ]
>
> [ Attachment, skipping... ]
>
>
>> --
>> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgsql-hackers
>>
>
>
--
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/
From | Date | Subject | |
---|---|---|---|
Next Message | Greg Smith | 2010-04-13 05:20:08 | Re: GSoC - proposal - Materialized Views in PostgreSQL |
Previous Message | tomas | 2010-04-13 04:28:04 | Re: debugger question |