Thank you for looling this. At Mon, 31 Oct 2016 17:11:17 +0100, Daniel Gustafsson <dan...@yesql.se> wrote in <3fc648b5-2b7f-4585-9615-207a44b73...@yesql.se> > > On 27 Oct 2016, at 09:23, Kyotaro HORIGUCHI > > <horiguchi.kyot...@lab.ntt.co.jp> wrote: > > Perl scripts are to be messy, I believe. Anyway the duplicate > > check as been built into the sub print_radix_trees. Maybe the > > same check is needed by some plain map files but it would be just > > duplication for the maps having radix tree. > > I took a small stab at doing some cleaning of the Perl scripts, mainly around > using the more modern (well, modern as in +15 years old) form for open(..), > avoiding global filehandles for passing scalar references and enforcing use > strict. Some smaller typos and fixes were also included. It seems my Perl > has > become a bit rusty so I hope the changes make sense. The produced files are > identical with these patches applied, they are merely doing cleaning as > opposed > to bugfixing. > > The attached patches are against the 0001-0006 patches from Heikki and you in > this series of emails, the separation is intended to make them easier to read.
I'm not sure how the discussion about this goes, these patches makes me think about coding style of Perl. The distinction between executable script and library is by intention with an obscure basis. Existing scripts don't get less modification, but library uses more restricted scopes to get rid of the troubles caused by using global scopes. But I don't have a clear preference on that. The TAP test scripts takes OO notations but I'm not sure convutils.pl also be better to take the same notation. It would be rarely edited hereafter and won't gets grown any more. As far as I see the obvious bug fixes in the patchset are the following, - 0007: load_maptable fogets to close input file. - 0010: commment for load_maptables is wrong. - 0011: hash reference is incorrectly dereferenced All other fixes other than the above three seem to be styling or syntax-generation issues and I don't know whether any recommendation exists... regards, -- Kyotaro Horiguchi NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers