Applying the patch from CLI-135 to put the public addValue method back, even though it just throws an exception. This maintains binary compatibility, for some level of compatibility.

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

Author: hen

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

@@ -401,7 +401,7 @@ public int getArgs()
      * 
      * @param value is a/the value of this Option
      */
-    void addValue(String value)
+    void addValueForProcessing(String value)
     {
         switch (numberOfArgs)
         {
@@ -649,4 +649,11 @@ void clearValues() {
         this.values.clear();
     }
 
+    public boolean addValue(Object object) {
+        throw new UnsupportedOperationException(
+           "The addValue method is not intended for client use. " + 
+           "Subclasses should use the addValueForProcessing method instead. "
+           );
+    }
+
 }

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

@@ -258,7 +258,7 @@ private void processProperties(Properties properties)
                     {
                         try
                         {
-                            opt.addValue(value);
+                            opt.addValueForProcessing(value);
                         }
                         catch (RuntimeException exp)
                         {
@@ -340,7 +340,7 @@ public void processArgs(Option opt, ListIterator iter)
             // found a value
             try
             {
-                opt.addValue( Util.stripLeadingAndTrailingQuotes(str) );
+                opt.addValueForProcessing( Util.stripLeadingAndTrailingQuotes(str) );
             }
             catch (RuntimeException exp)
             {

src/test/org/apache/commons/cli/OptionTest.java

@@ -23,8 +23,18 @@
  */
 public class OptionTest extends TestCase {
 
+   private static class TestOption extends Option {
+        public TestOption(String opt, boolean hasArg, String description) throws IllegalArgumentException {
+            super(opt, hasArg, description);
+        }
+        public boolean addValue(String value) {
+            addValueForProcessing(value);
+            return true;
+        }
+   }
+
    public void testClear() {
-       Option option = new Option("x", true, "");
+       TestOption option = new TestOption("x", true, "");
        assertEquals(0, option.getValuesList().size());
        option.addValue("a");
        assertEquals(1, option.getValuesList().size());
@@ -34,8 +44,8 @@ public void testClear() {
 
     // See http://issues.apache.org/jira/browse/CLI-21
     public void testClone() throws CloneNotSupportedException {
-        Option a = new Option("a", true, "");
-        Option b = (Option) a.clone();
+        TestOption a = new TestOption("a", true, "");
+        TestOption b = (TestOption) a.clone();
         assertEquals(a, b);
         assertNotSame(a, b);
         a.setDescription("a");