Re: pgsql: Fix perltidy breaking perlcritic
Andrew Dunstan <andrew@dunslane.net>
From: Andrew Dunstan <andrew@dunslane.net>
To: John Naylor <john.naylor@enterprisedb.com>, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>,
pgsql-committers@lists.postgresql.org
Date: 2022-09-14T02:05:20Z
Lists: pgsql-hackers
On 2022-09-13 Tu 05:25, John Naylor wrote: > On Mon, Sep 12, 2022 at 4:54 PM Dagfinn Ilmari Mannsåker > <ilmari@ilmari.org> wrote: > >>> eval "\$hash_ref = $_"; ## no critic (ProhibitStringyEval) >> I didn't see this until it got committed, since I'm not subscribed to >> -committers, but I think it would be even better to rely on the fact >> that eval returns the value of the last expression in the string, which >> also gets rid of the ugly quoting and escaping, per the attached. > Hmm, interesting. I agree it's a slight stylistic improvement. I was trying to keep as close as possible to the original. cheers andrew -- Andrew Dunstan EDB: https://www.enterprisedb.com
Commits
-
Make eval statement naturally proof against perltidy
- b060f57791cf 16.0 landed
-
Fix perltidy breaking perlcritic
- 12d40d4a8d04 16.0 cited