On 24 January 2011 03:06, Andrew Dunstan <[email protected]> wrote: > It's doing the right thing. But it's really spaghetti code, generated by > a2p. Matt was just (rightly) offended by the $[ setting, IIRC. > > The point is that it's close to being totally unmaintainable. > > But if you think you can remedy it without rewriting it, go for it. > > I think minimum requirements would be: > > * avoid setting $[ > * use strict; > * comments saying what it's actually doing > > > We want to be in a situation where of it ever does break because of some > external change, we're not left having to wade through the crap to find out > how to fix it. > > cheers > > andrew >
Well here's an absolutely minimal patch to remove the $[ assignments
and fix up the array accesses.
I have tested that parse.pl produces identical output for both HEAD
and the 65 commits which touch ecpg.*
or gram.y since the introduction on check_rules.pl
git log 81a82a13^..HEAD -- src/backend/parser/gram.y
src/interfaces/ecpg/preproc/{ecpg.??*,*.pl}
I've also tested that the new check_rules.pl dies when I add an extra
unused rule to ecpg.addons.
I'll try and make a proper rewrite sometime this week.
What is the minimal perl version that we are requiring these days?
ecpg_preproc_minimal_perl_cleanup.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
