Login
I think Schwern mention the double periods appearing at the
authorJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Wed, 29 Oct 2008 08:45:01 +0000 (08:45 +0000)
committerJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Wed, 29 Oct 2008 08:45:01 +0000 (08:45 +0000)
end of some violation descriptions and/or explanations.  So
we strip out whatever punctuation the author provided, and
let the user define their own, using a format.

lib/Perl/Critic/Violation.pm

index 7b653cb..cda76f3 100644 (file)
@@ -58,6 +58,11 @@ sub new {
                 '3rd arg to Violation->new() must be a PPI::Element';
         }
     }
+    
+    # Strip punctuation.  These are controlled by the user via the
+    # formats.  He/She can use whatever makes sense to them.
+    not ref $expl and $expl =~ s/[.]+ \z//xms; # Could be an array ref
+    $desc =~ s/[.]+ \z//xms;
 
     #Create object
     my $self = bless {}, $class;