On 2024-07-02 Tu 8:55 AM, Dagfinn Ilmari Mannsåker wrote:
Relatedly, I also had a look at prohibiting unused regex captures (RegularExpressions::ProhibitUnusedCapture), which found a few real cases, but also lots of false positives in Catalog.pm, because it doesn't understand that %+ uses all named captures, so I won't propose a patch for that until that's fixed upstream (https://github.com/Perl-Critic/Perl-Critic/pull/1065).
We could mark Catalog.pm with a "## no critic (ProhibitUnusedCapture)" and then use the test elsewhere.
cheers andrew -- Andrew Dunstan EDB:https://www.enterprisedb.com
