Removed clone() method - it was incorrectly implemented, but more importantly there is no obvious reason for Option to be cloneable. This resolves #28482 and #29908

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

Author: hen

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

@@ -32,7 +32,7 @@
  * @author <a href="mailto:jstrachan@apache.org">James Strachan</a>
  * @version $Revision$
  */
-public class Option implements Cloneable {
+public class Option {
 
     /** constant that specifies the number of argument values has 
         not been specified */
@@ -552,23 +552,6 @@ public java.util.List getValuesList()
         return this.values;
     }
 
-    /**
-     * @return a copy of this Option
-     */
-    public Object clone()
-    {
-        Option option = new Option(getOpt(), getDescription());
-
-        option.setArgs(getArgs());
-        option.setOptionalArg(hasOptionalArg());
-        option.setRequired(isRequired());
-        option.setLongOpt(getLongOpt());
-        option.setType(getType());
-        option.setValueSeparator(getValueSeparator());
-
-        return option;
-    }
-
     /** 
      * Dump state, suitable for debugging.
      *