Re: Generate GUC tables from .dat file

On Thu, Aug 28, 2025 at 1:53 PM Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
> Ok, good suggestions. I addressed all those, and did another cleanup
> pass over the script. (The formatting is from pgperltidy.)

I have no further comments on v3.

On Fri, Aug 29, 2025 at 1:03 AM David E. Wheeler <david(at)justatheory(dot)com> wrote:
> And since I griped about Perl style previously, I made a pass over modernizing it a bit. One might argue it’s less clear, of course; there is less alignment of the printing than in the original.

I find the loop over @parse a lot less readable this way.

> * Use the /r regex return sequence to simplify dquote() (requires Perl 5.14, IIRC)

I think our perlcritic configuration would complain about the lack of
return statement.

> * Use {$fh} syntax to make file handle arguments clearer

With this I wonder why the variable looks different for `print` vs. `open`.

--
John Naylor
Amazon Web Services