You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2015/09/26 14:06:14 UTC
svn commit: r1705430 - in
/maven/plugins/trunk/maven-clean-plugin/src/main/java/org/apache/maven/plugin/clean:
CleanMojo.java Cleaner.java Fileset.java
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;