You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2014/02/22 00:36:02 UTC
svn commit: r1570752 -
/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleResults.java
Author: dennisl
Date: Fri Feb 21 23:36:02 2014
New Revision: 1570752
URL: http://svn.apache.org/r1570752
Log:
[MCHECKSTYLE-205] NPE in CheckstyleReportGenerator.doFilesSummary:654 version 2.11 regression
Submitted by: Bob Fields
Reviewed by: Dennis Lundberg
Modified:
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleResults.java
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleResults.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleResults.java?rev=1570752&r1=1570751&r2=1570752&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleResults.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleResults.java Fri Feb 21 23:36:02 2014
@@ -57,7 +57,10 @@ public class CheckstyleResults
else
{
violations = new LinkedList<AuditEvent>();
- this.files.put( file, violations );
+ if ( file != null )
+ {
+ this.files.put( file, violations );
+ }
}
return violations;
@@ -65,7 +68,10 @@ public class CheckstyleResults
public void setFileViolations( String file, List<AuditEvent> violations )
{
- this.files.put( file, violations );
+ if ( file != null )
+ {
+ this.files.put( file, violations );
+ }
}
public Map<String, List<AuditEvent>> getFiles()