ECPG check variables hidden by locals v2
Boszormenyi Zoltan <zb@cybertec.at>
From: Boszormenyi Zoltan <zb@cybertec.at>
To: Michael Meskes <meskes@postgresql.org>
Cc: PostgreSQL Hacker <pgsql-hackers@postgresql.org>
Date: 2010-04-01T08:16:10Z
Lists: pgsql-hackers
Attachments
- pg90-ecpg-warn-hidden-vars-2-ctxdiff.patch (text/x-patch) patch
- test2.pgc (text/plain)
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/