svn commit: r1705430 - in /maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean: CleanMojo.java Cleaner.java Fileset.java

[khmarbaise]

Author: khmarbaise
Date: Sat Sep 26 12:06:14 2015
New Revision: 1705430

URL: http://svn.apache.org/viewvc?rev=1705430&view=rev
Log:
Cleaned up checkstyle reported errors.

Modified:
    
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/CleanMojo.java
    
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Cleaner.java
    
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Fileset.java

Modified: 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/CleanMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/CleanMojo.java?rev=1705430&r1=1705429&r2=1705430&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/CleanMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/CleanMojo.java
 Sat Sep 26 12:06:14 2015
@@ -184,11 +184,11 @@ public class CleanMojo
 
         try
         {
-            for ( File directory : getDirectories() )
+            for ( File directoryItem : getDirectories() )
             {
-                if ( directory != null )
+                if ( directoryItem != null )
                 {
-                    cleaner.delete( directory, null, followSymLinks, 
failOnError, retryOnError );
+                    cleaner.delete( directoryItem, null, followSymLinks, 
failOnError, retryOnError );
                 }
             }
 

Modified: 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Cleaner.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Cleaner.java?rev=1705430&r1=1705429&r2=1705430&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Cleaner.java
 (original)
+++ 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Cleaner.java
 Sat Sep 26 12:06:14 2015
@@ -152,7 +152,7 @@ class Cleaner
                     String[] filenames = canonical.list();
                     if ( filenames != null )
                     {
-                        String prefix = ( pathname.length() > 0 ) ? pathname + 
File.separatorChar : "";
+                        String prefix = pathname.length() > 0 ? pathname + 
File.separatorChar : "";
                         for ( int i = filenames.length - 1; i >= 0; i-- )
                         {
                             String filename = filenames[i];

Modified: 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Fileset.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Fileset.java?rev=1705430&r1=1705429&r2=1705430&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Fileset.java
 (original)
+++ 
maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean/Fileset.java
 Sat Sep 26 12:06:14 2015
@@ -45,26 +45,41 @@ public class Fileset
 
     private boolean useDefaultExcludes;
 
+    /**
+     * @return {@link #directory}
+     */
     public File getDirectory()
     {
         return directory;
     }
 
+    /**
+     * @return {@link #includes}
+     */
     public String[] getIncludes()
     {
         return ( includes != null ) ? includes : new String[0];
     }
 
+    /**
+     * @return {@link #excludes}
+     */
     public String[] getExcludes()
     {
         return ( excludes != null ) ? excludes : new String[0];
     }
 
+    /**
+     * @return {@link #followSymlinks}
+     */
     public boolean isFollowSymlinks()
     {
         return followSymlinks;
     }
 
+    /**
+     * @return {@link #useDefaultExcludes}
+     */
     public boolean isUseDefaultExcludes()
     {
         return useDefaultExcludes;