On 28.08.25 15:29, Daniel Gustafsson wrote:
On 28 Aug 2025, at 08:53, Peter Eisentraut <[email protected]> wrote:
I addressed all those, and did another cleanup pass over the script. (The
formatting is from pgperltidy.)
A tiny nitpick is that all the other generator scripts (that I looked at) in
the tree use GetOptions() with named parameter rather than dereference ARGV
directly:
+my $input_fname = $ARGV[0];
+my $output_fname = $ARGV[1];
I didn't use GetOptions() but changed it a bit per David Wheeler's
suggestion downthread.
Also, I would have directed the reader to guc_parameters.dat in the below
comment since that's the canonical copy of the default value, but it's a
personal preference as it can just as easily be argued for keeping guc_tables.c
I updated this comment and another one.
(There are a few more comments referring to guc_tables.c, which I think
are mostly still correct. I expect some of this will also be changed in
the near future. One change for example could be to generate the enum
declarations.)
