Subject: | Simplify StatementSequence and PodSequence usage |
Date: | Tue, 3 Jul 2018 17:44:03 -0700 |
To: | bug-ppr [...] rt.cpan.org |
From: | Adriano Ferreira <a.r.ferreira [...] gmail.com> |
At the latest release, 0.000021 <https://metacpan.org/release/PPR>, one of
the changes you added looked like this:
@@ -66,7 +66,7 @@ our $GRAMMAR = qr{
(?<PerlDocument> (?<PerlStdDocument>
\x{FEFF}?+ # Optional BOM marker
- (?&PerlStatementSequence)?+
+ (?&PerlStatementSequence)
)) # End of rule
(?<PerlStatementSequence> (?<PerlStdStatementSequence>
That made remember that PerlStatementSequence and PerlPodSequence match
sequences of zero or more "things", and thus may get away without the ?+
quantifier (which is implied). If that holds true, you may consider the
attached patch that applies this simplification throughout the code.
Message body is not shown because sender requested not to inline it.