Author: peterreilly
Date: Sun Aug 12 11:12:52 2007
New Revision: 565094

URL: http://svn.apache.org/viewvc?view=rev&rev=565094
Log:
checkstyle merge

Modified:
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/FilterSet.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/PatternSet.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/Permissions.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/ZipScanner.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/resources/Last.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/DeweyDecimal.java
    
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/ScriptRunnerHelper.java

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/FilterSet.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/FilterSet.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/FilterSet.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/FilterSet.java
 Sun Aug 12 11:12:52 2007
@@ -370,7 +370,7 @@
               }
            } catch (Exception ex) {
               throw new BuildException("Could not read filters from file: "
-                  + filtersFile,ex);
+                  + filtersFile, ex);
            } finally {
               FileUtils.close(in);
            }

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/PatternSet.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/PatternSet.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/PatternSet.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/PatternSet.java
 Sun Aug 12 11:12:52 2007
@@ -143,7 +143,7 @@
         }
     }
 
-    private static class InvertedPatternSet extends PatternSet {
+    private static final class InvertedPatternSet extends PatternSet {
         private InvertedPatternSet(PatternSet p) {
             setProject(p.getProject());
             addConfiguredPatternset(p);
@@ -524,7 +524,7 @@
 
     /**
      * Add an inverted patternset.
-     *
+     * @param p the pattern to invert and add.
      */
     public void addConfiguredInvert(PatternSet p) {
         addConfiguredPatternset(new InvertedPatternSet(p));

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/Permissions.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/Permissions.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/Permissions.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/Permissions.java
 Sun Aug 12 11:12:52 2007
@@ -53,7 +53,7 @@
     private boolean delegateToOldSM;
 
     // Mandatory constructor for permission object.
-    private static final Class[] PARAMS = { String.class, String.class };
+    private static final Class[] PARAMS = {String.class, String.class};
 
     /**
      * Create a set of Permissions.  Equivalent to calling

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/ZipScanner.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/ZipScanner.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/ZipScanner.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/ZipScanner.java
 Sun Aug 12 11:12:52 2007
@@ -27,7 +27,6 @@
 import org.apache.tools.ant.BuildException;
 import org.apache.tools.ant.types.resources.FileResource;
 import org.apache.tools.ant.types.resources.ZipResource;
-import org.apache.tools.ant.util.FileUtils;
 import org.apache.tools.zip.ZipEntry;
 import org.apache.tools.zip.ZipFile;
 

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/resources/Last.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/resources/Last.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/resources/Last.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/resources/Last.java
 Sun Aug 12 11:12:52 2007
@@ -42,7 +42,9 @@
         int i = count;
         Iterator iter = rc.iterator();
         int size = rc.size();
-        for (; i < size; i++) iter.next();
+        for (; i < size; i++) {
+            iter.next();
+        }
 
         ArrayList al = new ArrayList(count);
         for (; iter.hasNext(); i++) {

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
 Sun Aug 12 11:12:52 2007
@@ -124,6 +124,9 @@
             strIdxStart++;
         }
 
+        // CheckStyle:SimplifyBooleanReturnCheck OFF
+        // Check turned off as the code needs the comments for the various
+        // code paths.
         if (strIdxStart > strIdxEnd) {
             // String is exhausted
             return true;

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/DeweyDecimal.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/DeweyDecimal.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/DeweyDecimal.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/DeweyDecimal.java
 Sun Aug 12 11:12:52 2007
@@ -18,7 +18,6 @@
 package org.apache.tools.ant.util;
 
 import java.util.StringTokenizer;
-import java.util.Arrays;
 
 /**
  * Utility class to contain version numbers in "Dewey Decimal"

Modified: 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/ScriptRunnerHelper.java
URL: 
http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/ScriptRunnerHelper.java?view=diff&rev=565094&r1=565093&r2=565094
==============================================================================
--- 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/ScriptRunnerHelper.java
 (original)
+++ 
ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/util/ScriptRunnerHelper.java
 Sun Aug 12 11:12:52 2007
@@ -59,7 +59,7 @@
         if (text != null) {
             runner.addText(text);
         }
-        if (resources !=null) {
+        if (resources != null) {
             runner.loadResources(resources);
         }
         if (setBeans) {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to