Fixed PosixParser to stop bursting tokens when a non option character is found (CLI-163)

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

Author: ebourg

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

@@ -14,11 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.apache.commons.cli;
 
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Iterator;
+import java.util.List;
 
 /**
  * The class PosixParser provides an implementation of the 
@@ -31,7 +33,7 @@
 public class PosixParser extends Parser {
 
     /** holder for flattened tokens */
-    private ArrayList tokens = new ArrayList();
+    private List tokens = new ArrayList();
 
     /** specifies if bursting should continue */
     private boolean eatTheRest;
@@ -43,9 +45,9 @@ public class PosixParser extends Parser {
     private Options options;
 
     /**
-     * <p>Resets the members to their original state i.e. remove
+     * Resets the members to their original state i.e. remove
      * all of <code>tokens</code> entries, set <code>eatTheRest</code>
-     * to false and set <code>currentOption</code> to null.</p>
+     * to false and set <code>currentOption</code> to null.
      */
     private void init()
     {
@@ -92,30 +94,27 @@ private void init()
      * when an non option is found.
      * @return The flattened <code>arguments</code> String array.
      */
-    protected String[] flatten(Options options, String[] arguments, 
-                               boolean stopAtNonOption)
+    protected String[] flatten(Options options, String[] arguments, boolean stopAtNonOption)
     {
         init();
         this.options = options;
 
         // an iterator for the command line tokens
         Iterator iter = Arrays.asList(arguments).iterator();
-        String token;
 
         // process each command line token
         while (iter.hasNext())
         {
             // get the next command line token
-            token = (String) iter.next();
+            String token = (String) iter.next();
 
             // handle SPECIAL TOKEN
             if (token.startsWith("--"))
             {
                 if (token.indexOf('=') != -1)
                 {
                     tokens.add(token.substring(0, token.indexOf('=')));
-                    tokens.add(token.substring(token.indexOf('=') + 1, 
-                                               token.length()));
+                    tokens.add(token.substring(token.indexOf('=') + 1, token.length()));
                 }
                 else
                 {
@@ -136,8 +135,9 @@ else if (token.startsWith("-"))
                 {
                     processOptionToken(token, stopAtNonOption);
                 }
-                else if (options.hasOption(token)) {
-                	tokens.add(token);
+                else if (options.hasOption(token))
+                {
+                    tokens.add(token);
                 }
                 // requires bursting
                 else
@@ -164,7 +164,7 @@ else if (options.hasOption(token)) {
     }
 
     /**
-     * <p>Adds the remaining tokens to the processed tokens list.</p>
+     * Adds the remaining tokens to the processed tokens list.
      *
      * @param iter An iterator over the remaining tokens
      */
@@ -183,8 +183,10 @@ private void gobble(Iterator iter)
      * <p>If there is a current option and it can have an argument
      * value then add the token to the processed tokens list and 
      * set the current option to null.</p>
+     *
      * <p>If there is a current option and it can have argument
      * values then add the token to the processed tokens list.</p>
+     *
      * <p>If there is not a current option add the special token
      * "<b>--</b>" and the current <code>value</code> to the processed
      * tokens list.  The add all the remaining <code>argument</code>
@@ -194,7 +196,7 @@ private void gobble(Iterator iter)
      */
     private void process(String value)
     {
-        if ((currentOption != null) && currentOption.hasArg())
+        if (currentOption != null && currentOption.hasArg())
         {
             if (currentOption.hasArg())
             {
@@ -215,8 +217,8 @@ else if (currentOption.hasArgs())
     }
 
     /**
-     * <p>If it is a hyphen then add the hyphen directly to
-     * the processed tokens list.</p>
+     * If it is a hyphen then add the hyphen directly to
+     * the processed tokens list.
      *
      * @param hyphen The hyphen token
      */
@@ -229,6 +231,7 @@ private void processSingleHyphen(String hyphen)
      * <p>If an {@link Option} exists for <code>token</code> then
      * set the current option and add the token to the processed 
      * list.</p>
+     *
      * <p>If an {@link Option} does not exist and <code>stopAtNonOption</code>
      * is set then ignore the current token and add the remaining tokens
      * to the processed tokens list directly.</p>
@@ -278,14 +281,11 @@ else if (stopAtNonOption)
      */
     protected void burstToken(String token, boolean stopAtNonOption)
     {
-        int tokenLength = token.length();
-
-        for (int i = 1; i < tokenLength; i++)
+        for (int i = 1; i < token.length(); i++)
         {
             String ch = String.valueOf(token.charAt(i));
-            boolean hasOption = options.hasOption(ch);
 
-            if (hasOption)
+            if (options.hasOption(ch))
             {
                 tokens.add("-" + ch);
                 currentOption = options.getOption(ch);
@@ -300,6 +300,7 @@ protected void burstToken(String token, boolean stopAtNonOption)
             else if (stopAtNonOption)
             {
                 process(token.substring(i));
+                break;
             }
             else
             {

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

@@ -130,6 +130,18 @@ public void testStop() throws Exception
         assertTrue( "Confirm  2 extra args: " + cl.getArgList().size(), cl.getArgList().size() == 2);
     }
 
+    public void testStopBursting() throws Exception
+    {
+        String[] args = new String[] { "-azc" };
+
+        CommandLine cl = parser.parse(options, args, true);
+        assertTrue( "Confirm -a is set", cl.hasOption("a") );
+        assertFalse( "Confirm -c is not set", cl.hasOption("c") );
+
+        assertTrue( "Confirm  1 extra arg: " + cl.getArgList().size(), cl.getArgList().size() == 1);
+        assertTrue(cl.getArgList().contains("zc"));
+    }
+
     public void testMultiple() throws Exception
     {
         String[] args = new String[] { "-c",
@@ -155,8 +167,7 @@ public void testMultipleWithLong() throws Exception
                                        "foobar",
                                        "--bfile", "toast" };
 
-        CommandLine cl = parser.parse(options,args,
-                                        true);
+        CommandLine cl = parser.parse(options,args, true);
         assertTrue( "Confirm -c is set", cl.hasOption("c") );
         assertTrue( "Confirm  3 extra args: " + cl.getArgList().size(), cl.getArgList().size() == 3);