Fix JavaDocs comments

Fix self closing tags, invalid taglets, parameters typos, etc

Author: plamentotev

src/main/java/org/codehaus/plexus/archiver/AbstractUnArchiver.java

@@ -35,9 +35,10 @@
 import org.codehaus.plexus.util.IOUtil;
 import org.codehaus.plexus.util.StringUtils;
 
+// TODO there should really be constructors which take the source file.
+
 /**
  * @author <a href="mailto:[email protected]">Emmanuel Venisse</a>
- * @todo there should really be constructors which take the source file.
  */
 public abstract class AbstractUnArchiver
     extends AbstractLogEnabled

src/main/java/org/codehaus/plexus/archiver/ArchiveFileFilter.java

@@ -1,5 +1,7 @@
 package org.codehaus.plexus.archiver;
 
+import org.codehaus.plexus.components.io.fileselectors.FileSelector;
+
 import java.io.InputStream;
 
 /**

src/main/java/org/codehaus/plexus/archiver/Archiver.java

@@ -353,13 +353,13 @@ ResourceIterator getResources()
     String getDuplicateBehavior();
 
     /**
-     * Set the behavior of this archiver when duplicate files are detected. One of: <br/>
+     * Set the behavior of this archiver when duplicate files are detected. One of: <br>
      * <ul>
      * <li>add - Add the duplicates to the archive as duplicate entries</li>
      * <li>skip/preserve - Leave the first entry encountered in the archive, skip the new one</li>
      * <li>fail - throw an {@link ArchiverException}</li>
      * </ul>
-     * <br/>
+     * <br>
      * See {@link Archiver#DUPLICATES_ADD}, {@link Archiver#DUPLICATES_SKIP}, {@link Archiver#DUPLICATES_PRESERVE},
      * {@link Archiver#DUPLICATES_FAIL}.
      */

src/main/java/org/codehaus/plexus/archiver/jar/JarArchiver.java

@@ -264,9 +264,9 @@ private Manifest getManifest( InputStream is )
      * other specified manifests.
      * "mergewithoutmain" merges everything but the Main section of the manifests.
      * Default value is "skip".
-     * <p/>
+     * <p>
      * Note: if this attribute's value is not "skip", the created jar will not
-     * be readable by using java.util.jar.JarInputStream
+     * be readable by using java.util.jar.JarInputStream</p>
      *
      * @param config setting for found manifest behavior.
      */
@@ -692,18 +692,15 @@ protected final void writeIndexLikeList( List<String> dirs, List<String> files,
 
     /**
      * try to guess the name of the given file.
-     * <p/>
      * <p>
      * If this jar has a classpath attribute in its manifest, we
      * can assume that it will only require an index of jars listed
      * there. try to find which classpath entry is most likely the
      * one the given file name points to.</p>
-     * <p/>
      * <p>
      * In the absence of a classpath attribute, assume the other
      * files will be placed inside the same directory as this jar and
      * use their basename.</p>
-     * <p/>
      * <p>
      * if there is a classpath and the given file doesn't match any
      * of its entries, return null.</p>

src/main/java/org/codehaus/plexus/archiver/jar/Manifest.java

@@ -36,16 +36,16 @@
 
 /**
  * Holds the data of a jar manifest.
- * <p/>
+ * <p>
  * Manifests are processed according to the
- * {@link <a href="http://java.sun.com/j2se/1.4/docs/guide/jar/jar.html">Jar
- * file specification.</a>}.
+ * <a href="http://java.sun.com/j2se/1.4/docs/guide/jar/jar.html">Jar
+ * file specification.</a>
  * Specifically, a manifest element consists of
  * a set of attributes and sections. These sections in turn may contain
  * attributes. Note in particular that this may result in manifest lines
  * greater than 72 bytes (including line break) being wrapped and continued
  * on the next line. If an application can not handle the continuation
- * mechanism, it is a defect in the application, not this task.
+ * mechanism, it is a defect in the application, not this task.</p>
  *
  * @since Ant 1.4
  */

src/main/java/org/codehaus/plexus/archiver/tar/TarArchiver.java

@@ -64,7 +64,7 @@ public class TarArchiver
     /**
      * Set how to handle long files, those with a path>100 chars.
      * Optional, default=warn.
-     * <p/>
+     * <p>
      * Allowable values are
      * <ul>
      * <li> truncate - paths are truncated to the maximum length
@@ -75,6 +75,7 @@ public class TarArchiver
      * <li> posixwarn - posix extensions are used (with warning) for any paths greater than the maximum.
      * <li> omit - paths greater than the maximum are omitted from the archive
      * </ul>
+     * </p>
      *
      * @param mode the mode to handle long file names.
      */

src/main/java/org/codehaus/plexus/archiver/tar/TarUnArchiver.java

@@ -56,7 +56,7 @@ public TarUnArchiver( File sourceFile )
 
     /**
      * Set decompression algorithm to use; default=none.
-     * <p/>
+     * <p>
      * Allowable values are
      * <ul>
      * <li>none - no compression</li>
@@ -65,6 +65,7 @@ public TarUnArchiver( File sourceFile )
      * <li>snappy - Snappy compression</li>
      * <li>xz - Xz compression</li>
      * </ul>
+     * </p>
      *
      * @param method compression method
      */

src/main/java/org/codehaus/plexus/archiver/war/WarArchiver.java

@@ -28,7 +28,7 @@
  * An extension of <jar> to create a WAR archive.
  * Contains special treatment for files that should end up in the
  * <code>WEB-INF/lib</code>, <code>WEB-INF/classes</code> or
- * <code>WEB-INF</code> directories of the Web Application Archive.</p>
+ * <code>WEB-INF</code> directories of the Web Application Archive.
  * <p>
  * (The War task is a shortcut for specifying the particular layout of a WAR file.
  * The same thing can be accomplished by using the <i>prefix</i> and <i>fullpath</i>
@@ -72,7 +72,7 @@ public void setIgnoreWebxml( boolean excpectWebXml )
     /**
      * Indicates if the client is required to supply web.xml
      *
-     * @param excpectWebXml true if web xml is *expected* from the client
+     * @param expectWebXml true if web xml is *expected* from the client
      */
     public void setExpectWebXml( boolean expectWebXml )
     {

src/main/java/org/codehaus/plexus/archiver/zip/AbstractZipArchiver.java

@@ -480,7 +480,6 @@ protected void zipFile( InputStreamSupplier in, ConcurrentJarCreator zOut, Strin
 
     /**
      * Method that gets called when adding from java.io.File instances.
-     * <p/>
      * <p>
      * This implementation delegates to the six-arg version.</p>
      *
@@ -694,11 +693,9 @@ protected boolean createEmptyZip( File zipFile )
 
     /**
      * Do any clean up necessary to allow this instance to be used again.
-     * <p/>
      * <p>
      * When we get here, the Zip file has been closed and all we
      * need to do is to reset some globals.</p>
-     * <p/>
      * <p>
      * This method will only reset globals that have been changed
      * during execute(), it will not alter the attributes or nested

src/main/java/org/codehaus/plexus/archiver/zip/AbstractZipUnArchiver.java

@@ -55,7 +55,6 @@ public AbstractZipUnArchiver( final File sourceFile )
 
     /**
      * Sets the encoding to assume for file names and comments.
-     * <p/>
      * <p>
      * Set to <code>native-encoding</code> if you want your platform's native encoding, defaults to UTF8.
      * </p>