And here you go
Patch applies cleany, make check ok. Looks pretty good.
A minor detail I have just noticed, sorry: now that options are discarded
by functions, some string variable declarations should be moved back
inside the active branch. You moved them out because you where sharing the
variables between the active & inactive branches, but this is no longer
necessary, and the project practice seems to declare variables just where
they are needed. That would be pattern in d, encoding in encoding, fname
in f and g and include and out and s, prefix in gset, opt in help, opt* in
lo and pset and set, arg* in prompt, env* in setenv... and maybe a few
others.
--
Fabien.
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
