Checkstyle warning: variable order, complete javadoc.

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@1439807 13f79535-47bb-0310-9956-ffa450edef68

Author: netomi

src/main/java/org/apache/commons/cli/Option.java

@@ -38,15 +38,15 @@
  */
 public class Option implements Cloneable, Serializable
 {
-    /** The serial version UID. */
-    private static final long serialVersionUID = 1L;
-
     /** constant that specifies the number of argument values has not been specified */
     public static final int UNINITIALIZED = -1;
 
     /** constant that specifies the number of argument values is infinite */
     public static final int UNLIMITED_VALUES = -2;
 
+    /** The serial version UID. */
+    private static final long serialVersionUID = 1L;
+
     /** the name of the option */
     private String opt;
 
@@ -662,10 +662,14 @@ void clearValues()
 
     /**
      * This method is not intended to be used. It was a piece of internal 
-     * API that was made public in 1.0. It currently throws an UnsupportedOperationException. 
+     * API that was made public in 1.0. It currently throws an UnsupportedOperationException.
+     *
+     * @param value the value to add
+     * @return always throws an {@link UnsupportedOperationException}
+     * @throws UnsupportedOperationException always
      * @deprecated
-     * @throws UnsupportedOperationException
      */
+    @Deprecated
     public boolean addValue(String value)
     {
         throw new UnsupportedOperationException("The addValue method is not intended for client use. "