Re: pgsql: Fix perltidy breaking perlcritic - Mailing list pgsql-committers

From Andrew Dunstan
Subject Re: pgsql: Fix perltidy breaking perlcritic
Date
Msg-id cd7c9d7d-53bf-1a8f-669a-4c5ec6c601e6@dunslane.net
Whole thread Raw
In response to pgsql: Fix perltidy breaking perlcritic  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
List pgsql-committers
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




pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: pgsql: aix: No need to use mkldexport when we want to export all symbol
Next
From: Andres Freund
Date:
Subject: Re: pgsql: aix: No need to use mkldexport when we want to export all symbol