Author: dkulp
Date: Mon Mar 12 09:34:47 2007
New Revision: 517275

URL: http://svn.apache.org/viewvc?view=rev&rev=517275
Log:
Fix problem with not actually using suppression file


Modified:
    
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java

Modified: 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java?view=diff&rev=517275&r1=517274&r2=517275
==============================================================================
--- 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java
 (original)
+++ 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java
 Mon Mar 12 09:34:47 2007
@@ -571,6 +571,7 @@
                 CheckstyleResults results;
 
                 moduleFactory = getModuleFactory();
+                
                 config = ConfigurationLoader.loadConfiguration( configFile,
                                                                 new 
PropertiesExpander( overridingProperties ) );
                 results = executeCheckstyle( config, moduleFactory );
@@ -776,7 +777,7 @@
         }
 
         FilterSet filterSet = getSuppressions();
-
+        
         Checker checker = new Checker();
 
         // setup classloader, needed to avoid "Unable to get class information
@@ -1090,15 +1091,10 @@
     private FilterSet getSuppressions()
         throws MavenReportException
     {
-        if ( suppressionsFileExpression != null )
-        {
-            return null;
-        }
-
         try
         {
             File suppressionsFile = locator.resolveLocation( 
suppressionsLocation, "checkstyle-suppressions.xml" );
-
+            
             if ( suppressionsFile == null )
             {
                 return null;


Reply via email to