diff options
author | Guillem Jover <guillem@debian.org> | 2014-05-06 01:33:17 +0200 |
---|---|---|
committer | Guillem Jover <guillem@debian.org> | 2014-05-17 13:30:52 +0200 |
commit | 1e5163809eff116ff1b34956562e1f6ec0bbf370 (patch) | |
tree | 006c0dee78f48137778ecf46aab43eb6d644ec6a /test | |
parent | 395f6ced94938c72927575e8d5b40305da321799 (diff) | |
download | dpkg-1e5163809eff116ff1b34956562e1f6ec0bbf370.tar.gz |
perl: Fix unused capture groups false positives
These are false positives, as when there's no match perl will leave
the subpattern variables untouched. We'll just use the opportunity to
unconfuse perlcritic, making some of those variable usages more clear
by using intermediate variables.
Fixes RegularExpressions::ProhibitUnusedCapture.
Warned-by: perlcritic
Diffstat (limited to 'test')
-rw-r--r-- | test/critic.t | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/test/critic.t b/test/critic.t index 10d47b048..aed2f1eb9 100644 --- a/test/critic.t +++ b/test/critic.t @@ -80,6 +80,7 @@ my @policies = qw( NamingConventions::Capitalization Objects::ProhibitIndirectSyntax RegularExpressions::ProhibitSingleCharAlternation + RegularExpressions::ProhibitUnusedCapture RegularExpressions::ProhibitUnusualDelimiters RegularExpressions::RequireBracesForMultiline Subroutines::ProhibitExplicitReturnUndef |