|
ID |
Subject |
Status |
Severity |
Last Updated |
Broken in |
Fixed in |
| 72980 |
ProhibitMagicNumbers false positive with capture variables |
open |
Normal |
6 years ago |
1.116 |
|
| 94087 |
t/20_policies.t fail on 5.19.10 |
open |
|
7 years ago |
|
|
| 87032 |
wrong return precedence in Perl::Critic::Utils::interpolate |
open |
|
7 years ago |
|
1.120 |
| 89923 |
RequireLocalizedPunctuationVars should allow $DATA |
new |
Normal |
7 years ago |
1.118 |
|
| 89904 |
Perl::Critic::Policy::Subroutines::ProhibitReturnSort documentation question |
new |
Unimportant |
7 years ago |
1.120_01 |
|
| 89432 |
Loading Perl::Critic fails in PodSpelling |
new |
Important |
7 years ago |
1.119 |
|
| 89375 |
Test t/00_modules.t Failed test 'use Perl::Critic::Policy::Documentation::PodSpelling;' |
new |
|
7 years ago |
|
|
| 89190 |
InputOutput::RequireBriefOpen false positive |
new |
|
7 years ago |
1.119 |
|
| 89097 |
RegularExpressions::ProhibitCaptureWithoutTest should allow $foo = $1 if (condition with regex) |
new |
|
7 years ago |
|
|
| 89096 |
ErrorHandling::RequireCarping should allow die; and die $@; |
new |
|
7 years ago |
|
|
| 87822 |
Miscellanea::RequireRcsKeywords is advertised, but missing. |
open |
|
7 years ago |
1.118 |
|
| 86912 |
local $_ uninitialized |
open |
Wishlist |
7 years ago |
|
|
| 86083 |
Perl::Critic::Policy::Subroutines::ProhibitBuiltinHomonyms doesn't know about :method |
open |
Wishlist |
7 years ago |
1.118 |
|
| 82965 |
-1 is not one of the allowed literal values (0, 1, 2). Use the Readonly or Const::Fast module or the "constant" pragma instead |
open |
Normal |
7 years ago |
|
|
| 88031 |
Suggested policy: use modification operators like += |
new |
Wishlist |
7 years ago |
|
|
| 86604 |
typo fixes |
new |
Unimportant |
7 years ago |
1.118 |
|
| 88473 |
"use constant" Inconsistency |
new |
|
7 years ago |
|
|
| 88290 |
RequireCheckedSyscalls false positive for opening from a string |
new |
Normal |
7 years ago |
1.118 |
|
| 67659 |
Named Captures (ProhibitUnusedCapture) |
open |
Normal |
7 years ago |
1.115 |
|
| 68783 |
RequireBriefOpen vs sub |
open |
|
7 years ago |
|
|
| 88007 |
Suggested policy: redundant () in postfix control flow |
new |
|
7 years ago |
|
|
| 87441 |
ProhibitAugmentedAssignmentInDeclaration should (optionally) allow the: our $foo ||= ...; idiom |
new |
|
7 years ago |
|
|
| 86973 |
Migrate from svn to github |
open |
|
7 years ago |
|
|
| 87369 |
Migrate issues from RT to github |
new |
Important |
7 years ago |
|
|
| 86942 |
Is Perl::Critic being maintained? |
open |
|
7 years ago |
|
|
| 39337 |
P::C::Policy::RegularExpressions::ProhibitCaptureWithoutTest does not recognize the trinary conditional as a test... |
open |
Normal |
7 years ago |
1.092 |
|
| 86913 |
Subroutines::ProtectPrivateSubs and Encode::_utf8_off |
new |
Wishlist |
7 years ago |
|
|
| 86826 |
Enhancement: policy to detect 'for' list modification inside loop |
new |
|
7 years ago |
|
|
| 86616 |
Enhancement: Policy to detect non-transitive comparison functions |
open |
Wishlist |
7 years ago |
|
|
| 86208 |
Patch for _is_backup in Perl::Critic::Utils |
new |
|
8 years ago |
|
|
| 85938 |
Possible Bug in Perl::Critic::Policy::Subroutines::RequireFinalReturn |
new |
|
8 years ago |
|
|
| 84898 |
libperl-critic-perl: until block false-positive in hash key |
new |
|
8 years ago |
|
|
| 84945 |
Allow Perl::Critic::Policy::CodeLayout::RequireTidyCode to configure source formatter class |
new |
Wishlist |
8 years ago |
1.118 |
|
| 84907 |
RegularExpressions::ProhibitEnumeratedClasses encourages \d over [0-9] |
new |
|
8 years ago |
|
|
| 84135 |
Perl::Critic::Policy::Modules::RequireVersionVar should not complain in test files |
open |
|
8 years ago |
|
|
| 84734 |
Suggested policy: while (<$fh>) inside subroutine |
new |
|
8 years ago |
|
|
| 83726 |
libperl-critic-perl: eval return value checking policy now obsolete |
new |
|
8 years ago |
|
|
| 79200 |
Subroutines::RequireArgUnpacking allow_delegation_to does not handle @_ plus additional args |
open |
Wishlist |
8 years ago |
|
|
| 83446 |
Perl::Critic::Utils::all_perl_files option to skip symlinks |
new |
Wishlist |
8 years ago |
1.118 |
|
| 82813 |
Enhancement with Patch Attached: Add CERT Theme |
open |
|
8 years ago |
|
|
| 82966 |
Use the Readonly or Const::Fast module when comparing with Some::Module::VERSION |
open |
|
8 years ago |
|
|
| 82636 |
libperl-critic-perl: Mixed bool operators false positive on postfix control |
open |
|
8 years ago |
|
|
| 82921 |
Deprecated modules are not EvilModules |
open |
Normal |
8 years ago |
|
|
| 82660 |
libperl-critic-perl: Comma separated statement false positive on scalar hashref |
new |
|
8 years ago |
|
|
| 82177 |
[PATCH] a policy to prohibit unless-else |
open |
|
8 years ago |
|
|
| 82415 |
uninitialized values on particular sample file |
new |
|
8 years ago |
|
|
| 82321 |
libperl-critic-perl: Documentation::RequirePodLinksIncludeText is obsolete |
open |
|
8 years ago |
|
|
| 82120 |
RegularExpressions::RequireExtendedFormatting: possibly false positive |
stalled |
Normal |
8 years ago |
1.118 |
|
| 80570 |
Suggested policy: double semicolon |
new |
Wishlist |
8 years ago |
1.118 |
|
| 80542 |
complexity in regular expressions |
new |
|
8 years ago |
|
|
| 80136 |
ProhibitUnreachableCode does not consider when conditional |
open |
Normal |
8 years ago |
1.118 |
|
| 80135 |
RequireFinalReturn does not consider ... by default |
new |
Normal |
8 years ago |
1.118 |
|
| 79644 |
All 13 standard Subroutine policies ignore anonymous subs |
open |
|
8 years ago |
|
|
| 79602 |
"open $fh,'<',$name || die" gets past InputOutput::RequireCheckedOpen |
new |
|
8 years ago |
|
|
| 79231 |
"return undef" policy |
open |
|
8 years ago |
|
|
| 79479 |
Reused variable name in lexical scope: $_ |
open |
Normal |
8 years ago |
1.118 |
|
| 79289 |
False Postive in Perl::Critic::Utils::is_in_void_context() |
patched |
Important |
8 years ago |
1.110_001 |
|
| 79138 |
RequireArgUnpacking confused by @_ in finally{} |
open |
|
8 years ago |
|
|
| 79162 |
Mechanism to ignore .tdy files |
open |
Wishlist |
8 years ago |
|
|
| 78722 |
Add Mouse Moo and Mo to modules that provide stict/warnings |
new |
Wishlist |
8 years ago |
|
|
| 75329 |
[PATCH] New policy: Modules::ProhibitInheritingExporter |
open |
Wishlist |
8 years ago |
1.117 |
|
| 75300 |
[PATCH] "use base 'Exporter'" => "use Exporter 'import'" |
open |
Wishlist |
8 years ago |
1.117 |
|
| 70763 |
Policy/ClassHierarchies/ProhibitExplicitISA.pm - add recommendation: use parent |
open |
Wishlist |
8 years ago |
1.116 |
|
| 67850 |
[PATCH] ProhibitLeadingZeros on shmget() |
new |
|
8 years ago |
|
|
| 77397 |
Suggested policy: avoid 'given' |
new |
Wishlist |
8 years ago |
1.117 |
|
| 77360 |
RequireUseWarnings doesn't accept "use 5.16.0" |
open |
|
8 years ago |
|
|
| 77693 |
[PATCH] Stop using obsolete make-local-hook function |
open |
|
9 years ago |
|
|
| 64776 |
Suggested policy: forbid .* at start or end of unanchored regexp |
open |
Wishlist |
9 years ago |
1.111 |
|
| 77214 |
Suggested policy: double comma |
new |
Wishlist |
9 years ago |
1.117 |
|
| 77143 |
Suggested policy: duplicate import |
new |
Wishlist |
9 years ago |
1.117 |
|
| 77085 |
Suggested policy: format string vulnerabilities |
new |
Wishlist |
9 years ago |
1.117 |
|
| 76327 |
Suggested policy: if string constant exists, check it is used consistently |
new |
Wishlist |
9 years ago |
1.117 |
|
| 75920 |
ProhibitInterpolationOfLiterals false positive with map and regexp match |
open |
Normal |
9 years ago |
1.117 |
|
| 75904 |
Suggested policy: iterate over arrays with 'each' instead of $i=0;$i++ |
new |
Wishlist |
9 years ago |
1.117 |
|
| 75794 |
Suggested policy: /r flag on substitutions |
open |
Wishlist |
9 years ago |
1.117 |
|
| 75765 |
Suggested policy: fold cases using fc |
open |
Wishlist |
9 years ago |
1.117 |
|
| 72660 |
Exclude pragmas from Modules::RequireExplicitPackage |
open |
|
9 years ago |
|
|
| 43908 |
Add MANIFEST to the repository |
open |
Unimportant |
9 years ago |
1.097_002 |
|
| 31624 |
Policy suggestion : identify similar/duplicate code for refactoring |
open |
Wishlist |
9 years ago |
1.080 |
|
| 70869 |
False Positive Package Declaration |
open |
Wishlist |
9 years ago |
|
|
| 41871 |
InputOutput::ProhibitReadlineInForLoop mistakes glob for readline |
open |
|
9 years ago |
1.092 |
|
| 72823 |
ProhibitMagicNumbers does not recognize octal in allowed_values |
open |
Normal |
9 years ago |
1.116 |
|
| 49636 |
ProhibitUselessNoCritic not of user-disabled policies |
open |
|
9 years ago |
|
|
| 72824 |
ProhibitReusedNames does not catch 'foreach my $x' |
open |
Normal |
9 years ago |
1.116 |
|
| 72737 |
Parsing bug for return statement |
open |
Normal |
9 years ago |
1.116 |
|
| 71168 |
New policy: Prohibit old-style smart match operator |
open |
Wishlist |
9 years ago |
|
|
| 70870 |
RequireUseWarnings ignores #! -w switch |
open |
|
9 years ago |
|
|
| 70422 |
exemptions for ValuesAndExpressions::ProhibitNoisyQuotes |
new |
Wishlist |
9 years ago |
1.116 |
|
| 69865 |
RegularExpressions::ProhibitCaptureWithoutTest prohibits local($1,$2,$3) |
new |
|
9 years ago |
|
|
| 69670 |
Perl::Critic::Policy::Documentation::RequirePodSections and core module Pod::Usage don't play well together out of the box. |
open |
|
9 years ago |
|
|
| 69234 |
new 5.014 package { }; syntax breaks many tests |
open |
|
9 years ago |
|
|
| 69232 |
Suggested policy: forbid test ($#foo > -1) |
open |
Wishlist |
9 years ago |
1.115 |
|
| 68785 |
RequireCheckingReturnValueOfEval and "." concats |
new |
|
10 years ago |
|
|
| 68364 |
RequireLocalizedPunctuationVars exempt $? |
open |
|
10 years ago |
|
|
| 65231 |
RequireTrailingCommas only at newline |
open |
Wishlist |
10 years ago |
|
|
| 68086 |
BEGIN { @ISA } detected as "run-time" |
open |
|
10 years ago |
|
|
| 65464 |
ProhibitUnusualDelimiters relax when // and {} in pattern |
open |
Wishlist |
10 years ago |
|
|
| 67491 |
Perlcritic reports problem when evaluating a file with embedded form feed chars. |
open |
|
10 years ago |
|
|
| 67499 |
"Complex Mappings" and "Comma used to separate statements" catches a one-liner |
open |
Unimportant |
10 years ago |
1.115 |
|
| 67256 |
ProhibitUnusedCapture false positive |
open |
Normal |
10 years ago |
1.115 |
|
| 65839 |
Allow VERSION declarations in BEGIN blocks prior to use strict; use warnings; |
open |
Normal |
10 years ago |
1.113 |
|
| 67212 |
CodeLayout::RequireTidyCode relative perltidyrc file |
open |
Wishlist |
10 years ago |
|
|
| 67142 |
Suggested policy: forbid /o regexp flag |
new |
Wishlist |
10 years ago |
1.114 |
|
| 48422 |
Allow flow control method calls in ControlStructures::ProhibitPostfixControls |
open |
Wishlist |
10 years ago |
1.102
1.113 |
|
| 28676 |
False positive in TestingAndDebugging::RequireUseStrict |
open |
Normal |
10 years ago |
1.061 |
|
| 65513 |
RequireLocalizedPunctuationVars should allow $0 |
open |
|
10 years ago |
|
|
| 65465 |
ProhibitUnusedVariables notice readline <$fh> |
open |
|
10 years ago |
|
|
| 65597 |
Another ProhibitCaptureWithoutTest false positive |
new |
Normal |
10 years ago |
1.111 |
|
| 65230 |
ProhibitTrailingWhitespace comments, strings, heredocs |
open |
|
10 years ago |
|
|
| 65220 |
ProhibitReusedNames vs "my" in expression |
open |
|
10 years ago |
|
|
| 65191 |
Suggested policy: forbid array on LHS of ~~ |
new |
Wishlist |
10 years ago |
1.111 |
|
| 64582 |
Perl::Critic::PolicyFactory shouldn't warn if profile disables non-installed policy (patch) |
open |
Wishlist |
10 years ago |
1.109 |
|
| 63965 |
Exception for special filehandles in InputOutput::RequireBracedFileHandleWithPrint |
new |
Wishlist |
10 years ago |
|
|
| 63904 |
bug in Perl-Critic - Thinks hash constructor is block |
stalled |
|
10 years ago |
|
|
| 63718 |
RegularExpressions::ProhibitFixedStringMatches could report on /foo/ as well |
open |
Wishlist |
10 years ago |
|
|
| 63472 |
Suggested policy: comma instead of semicolon |
open |
Wishlist |
10 years ago |
1.109 |
|
| 63252 |
Faster rewrite of Variables::ProhibitReusedNames |
open |
Wishlist |
10 years ago |
|
|
| 27654 |
False positive for ValuesAndExpressions::ProhibitCommaSeparatedStatements |
open |
|
10 years ago |
1.053 |
|
| 62377 |
Suggested policy: mkdir(x, 0777) is better as mkdir(x) |
open |
Wishlist |
10 years ago |
1.109 |
|
| 59176 |
"no critic" for pod after __END__ |
open |
|
10 years ago |
|
|
| 59175 |
policy object violation() method called in different package |
open |
Wishlist |
10 years ago |
|
|
| 43038 |
ProhibitExitInSubroutines versus blocks |
open |
|
10 years ago |
|
|
| 43457 |
Recognize 'Modern::Perl' |
open |
|
10 years ago |
1.096 |
|
| 60801 |
Extend Perl::Critic::Policy to support refactoring |
open |
Wishlist |
10 years ago |
|
|
| 59111 |
ProhibitLvalueSubstr docs |
new |
|
10 years ago |
|
|
| 58953 |
perlcritic shebang created incorrectly |
open |
|
10 years ago |
|
|
| 24887 |
Policy: Check for illegal illegal open source module licensing |
new |
Wishlist |
10 years ago |
|
|
| 57638 |
Subroutines::RequireFinalReturn should ignore constant subs |
open |
Normal |
11 years ago |
1.105 |
|
| 57807 |
Subroutines::ProhibitSubroutinePrototypes: add an exception for constant functions |
open |
Normal |
11 years ago |
1.106 |
|
| 57640 |
Subroutines::RequireFinalReturn should ignore DESTROY sub |
open |
Normal |
11 years ago |
1.105 |
|
| 57383 |
Using Module::Pluggable is problematic (needs to allow 'execpt' regex to avoid SCCS files when in source control context |
new |
|
11 years ago |
|
|
| 57065 |
ProhibitIndirectSyntax recognised by "Foo::" |
new |
|
11 years ago |
|
|
| 55750 |
ProhibitFixedStringMatches on qr in function call |
open |
|
11 years ago |
|
|
| 56615 |
POD manual page: Add POSIX compliant checks |
open |
Wishlist |
11 years ago |
|
|
| 56629 |
Quotes used with an empty string (false positive) |
open |
Normal |
11 years ago |
|
|
| 56624 |
Double-sigil dereference $% (possibly false positive) |
open |
Normal |
11 years ago |
|
|
| 38292 |
Suggested policy: forbid ^ and $ |
open |
Wishlist |
11 years ago |
|
|
| 38838 |
Suggested policy: say 'x' instead of print "x\n" |
open |
Wishlist |
11 years ago |
|
|
| 41872 |
Suggested policy: avoid builtin glob() except for simple cases |
new |
Wishlist |
11 years ago |
1.092 |
|
| 55751 |
ProhibitIndirectSyntax on dynaloader bootstrap() |
new |
Wishlist |
11 years ago |
|
|
| 55948 |
Documentation::PodSpelling report at first pod location |
new |
Wishlist |
11 years ago |
|
|
| 56621 |
Check the spelling in your POD: html (possibly false positive) |
open |
Wishlist |
11 years ago |
1.105 |
|
| 56788 |
ValuesAndExpressions::ProhibitMismatchedOperators: understand rand() |
new |
Wishlist |
11 years ago |
1.105 |
|
| 54764 |
ErrorHandling::RequireCheckingReturnValueOfEval false positive on 'return eval' |
new |
Normal |
11 years ago |
1.105 |
|
| 50894 |
RequireInitializationForLocalVars on $@ and $! |
open |
|
11 years ago |
|
|
| 50727 |
McCabe calcuation is too high (in PerlCritic) |
open |
|
11 years ago |
|
|
| 50080 |
new policy suggestion- prohibit unspecified context for called subroutines in lists |
new |
Wishlist |
11 years ago |
|
|
| 36129 |
BuiltinFunctions::ProhibitReverseSortBlock too strict |
open |
Normal |
11 years ago |
1.082 |
|
| 37886 |
CodeLayout::ProhibitQuotedWordLists should be only for words |
open |
Unimportant |
11 years ago |
1.090 |
|
| 48841 |
RegularExpressions::ProhibitSingleCharAlternation overzealous when some but not all alternatives are single chars |
open |
Normal |
11 years ago |
1.103 |
|
| 48219 |
no warning from Perl::Critic::Utils::is_script() please |
open |
|
11 years ago |
|
|
| 47572 |
ProhibitUselessNoCritic no report if nothing suppressed (patch) |
open |
|
11 years ago |
|
|
| 48033 |
False positive for ValuesAndExpressions::ProhibitCommaSeparatedStatements |
open |
|
11 years ago |
|
|
| 47358 |
BuiltinFunctions::ProhibitVoidMap is confusable about context, apparently. |
open |
|
11 years ago |
1.098 |
|
| 46971 |
ProhibitLocalVars on localized hash entry |
open |
|
11 years ago |
|
|
| 46980 |
RequireCheckingReturnValueOfEval allow test script ok() [patch] |
new |
|
12 years ago |
|
|
| 46727 |
RequireLocalizedPunctuationVars should not want to localize in top-level scope |
new |
Unimportant |
12 years ago |
1.098 |
|
| 45051 |
"return value of eval not tested" doesn't give PBP reference |
open |
Normal |
12 years ago |
1.098 |
|
| 44988 |
RequireLocalizedPunctuationVars allow array $a[0] |
open |
|
12 years ago |
|
|
| 44618 |
Add support for translated editions of PBP |
new |
Wishlist |
12 years ago |
1.098 |
|
| 44446 |
Error detecting magic variable (I think it's $&) |
open |
Normal |
12 years ago |
1.096 |
|
| 43909 |
COPYRIGHT and AUTHOR |
open |
Unimportant |
12 years ago |
|
|
| 42254 |
several errors (false-negative results) |
open |
|
12 years ago |
|
|
| 43754 |
ProhibitReusedNames vs "our" of multi-package file |
open |
|
12 years ago |
|
|
| 43651 |
False Positive in Perl::Critic::Policy::BuiltinFunctions::ProhibitVoidMap |
open |
|
12 years ago |
|
|
| 38289 |
Suggested policy: capture variable passed in subroutine call |
open |
Wishlist |
12 years ago |
|
|
| 23291 |
Policy suggestion: 80 columns |
open |
Wishlist |
12 years ago |
|
|
| 41715 |
CodeLayout::RequireConsistentNewlines omits filename |
open |
|
12 years ago |
1.092 |
|
| 37021 |
perlcritcrc settings ignored |
stalled |
|
12 years ago |
|
|
| 41938 |
Perl::Critic::Policy::Variables::ProhibitMatchVars exception |
open |
|
12 years ago |
1.093_03 |
|
| 38646 |
CodeLayout::ProhibitParensWithBuiltins exception when in ?: |
open |
Wishlist |
12 years ago |
|
|
| 38629 |
ErrorHandling::RequireCheckingReturnValueOfEval not allowing "return eval ..." |
new |
Normal |
12 years ago |
|
|
| 38074 |
Suggested policy: don't use the same module twice in the same source file |
open |
Wishlist |
12 years ago |
|
|
| 37905 |
do BLOCK while shouldn't be prohibited at ControlStructures::ProhibitPostfixControls |
open |
Normal |
12 years ago |
1.090 |
|
| 37885 |
ValuesAndExpressions::ProhibitCommaSeparatedStatements false positive |
open |
Normal |
12 years ago |
1.090 |
|
| 37850 |
ProhibitExplicitStdin doesn't quite match what PBP says |
new |
Normal |
12 years ago |
1.088 |
|
| 36123 |
InputOutput::RequireBracedFileHandleWithPrint confused by here document |
open |
|
12 years ago |
|
|
| 36249 |
anon hash ref treated as code block |
open |
|
13 years ago |
|
|
| 31978 |
"Use IO::Interactive::is_interactive() instead of -t" dubious? |
open |
Wishlist |
13 years ago |
|
|
| 36016 |
Prohibit numbered captures in Perl 5.10 |
open |
Wishlist |
13 years ago |
1.082 |
|
| 31972 |
PodSpelling/Open2 Kills Critic on Win32 |
open |
Critical |
13 years ago |
1.07
1.071
1.072
1.073
1.074
1.075_001
1.076
1.077
1.078
1.079_001
1.079_002
1.079_003
1.080 |
|
| 32013 |
octals and bitwise ops |
new |
Wishlist |
13 years ago |
|
|
| 20714 |
feature requests |
open |
|
14 years ago |
|
|
| 24618 |
Capture variable used outside conditional (Wrong) |
open |
Normal |
14 years ago |
1.01 |
|
| 21506 |
RequirePodSections still runs after global ## no critic |
stalled |
Important |
14 years ago |
0.2 |
|
| 20465 |
Magic <> should be a warning |
open |
|
14 years ago |
|
|
RT for rt.cpan.org