You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sn...@apache.org on 2008/02/16 15:48:02 UTC

svn commit: r628313 - in /maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war: AbstractWarMojo.java packaging/WarPackagingContext.java

Author: snicoll
Date: Sat Feb 16 06:48:01 2008
New Revision: 628313

URL: http://svn.apache.org/viewvc?rev=628313&view=rev
Log:
Updateed Javadoc.

Modified:
    maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/AbstractWarMojo.java
    maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/packaging/WarPackagingContext.java

Modified: maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/AbstractWarMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/AbstractWarMojo.java?rev=628313&r1=628312&r2=628313&view=diff
==============================================================================
--- maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/AbstractWarMojo.java (original)
+++ maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/AbstractWarMojo.java Sat Feb 16 06:48:01 2008
@@ -612,11 +612,8 @@
 
         public boolean isNonFilteredExtension( String fileName )
         {
-            if (StringUtils.isEmpty( fileName ))
-            {
-                return false;
-            }
-            return nonFilteredFileExtensions.contains( FileUtils.extension( fileName ) );
+            return !StringUtils.isEmpty( fileName ) &&
+                nonFilteredFileExtensions.contains( FileUtils.extension( fileName ) );
         }
         
     }

Modified: maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/packaging/WarPackagingContext.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/packaging/WarPackagingContext.java?rev=628313&r1=628312&r2=628313&view=diff
==============================================================================
--- maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/packaging/WarPackagingContext.java (original)
+++ maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/packaging/WarPackagingContext.java Sat Feb 16 06:48:01 2008
@@ -159,7 +159,9 @@
     List getOwnerIds();
     
     /**
-     * @return {@link MavenFileFilter}
+     * Returns the {@link MavenFileFilter} instance to use.
+     *
+     * @return the maven file filter to use
      * @since 2.1-alpha-2
      */
     MavenFileFilter getMavenFileFilter();
@@ -171,8 +173,12 @@
     List getFilterWrappers();
     
     /**
-     * @param fileName
-     * @return @since 2.1-alpha-2
+     * Specify if the given <tt>fileName</tt> belongs to the list of extensions
+     * that must not be filtered
+     *
+     * @param fileName the name of file
+     * @return <tt>true</tt> if it should not be filtered, <tt>false</tt> otherwise
+     * @since 2.1-alpha-2
      */
     boolean isNonFilteredExtension( String fileName );
 }