You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by fg...@apache.org on 2005/12/30 17:46:19 UTC
svn commit: r360118 -
/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java
Author: fgiust
Date: Fri Dec 30 08:46:16 2005
New Revision: 360118
URL: http://svn.apache.org/viewcvs?rev=360118&view=rev
Log:
don't add sections for files without warnings
Modified:
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java
URL: http://svn.apache.org/viewcvs/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java?rev=360118&r1=360117&r2=360118&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java Fri Dec 30 08:46:16 2005
@@ -520,7 +520,12 @@
for( Iterator files = results.getFiles().keySet().iterator(); files.hasNext(); )
{
String filename = (String) files.next();
- List violations = (List) results.getFileViolations( filename );
+ List violations = results.getFileViolations( filename );
+ if ( violations.isEmpty() )
+ {
+ // skip files without violations
+ continue;
+ }
sink.tableRow();
@@ -557,7 +562,13 @@
while ( files.hasNext() )
{
String file = (String) files.next();
- List violations = (List) results.getFileViolations( file );
+ List violations = results.getFileViolations( file );
+
+ if ( violations.isEmpty() )
+ {
+ // skip files without violations
+ continue;
+ }
doxiaHack();
sink.section1();