PatternOptionBuilder now supports required options

PR:10890
Submitted by:	Torsten Curdt
Reviewed by:	Rob Oxspring


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

Author: Robert James Oxspring

project.xml

@@ -78,7 +78,7 @@
 
     <dependency>
       <id>junit</id>
-      <version>3.7</version>
+      <version>3.8.1</version>
     </dependency>
 
   </dependencies>

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

@@ -117,7 +117,7 @@ public static boolean isValueCode(char ch)
     {
         if ((ch != '@') && (ch != ':') && (ch != '%') && (ch != '+')
             && (ch != '#') && (ch != '<') && (ch != '>') && (ch != '*')
-            && (ch != '/'))
+            && (ch != '/') && (ch != '!'))
         {
             return false;
         }

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

@@ -5,7 +5,7 @@
  * version 1.1, a copy of which has been included with this distribution in
  * the LICENSE file.
  * 
- * $Id: BugsTest.java,v 1.16 2003/01/17 20:00:14 jkeyes Exp $
+ * $Id: BugsTest.java,v 1.17 2004/03/26 19:35:01 roxspring Exp $
  */
 
 package org.apache.commons.cli;
@@ -383,5 +383,16 @@ public void test15648() throws Exception {
         CommandLine line = parser.parse( options, args );
         assertEquals( "Two Words", line.getOptionValue( "m" ) );
     }
+    
+    public void test27575() {
+    	Options options = PatternOptionBuilder.parsePattern("hc!<");
+    	assertNotNull(options);
+    	Option h = options.getOption("-h");
+    	assertNotNull(h);
+    	assertFalse(h.isRequired());
+    	Option c = options.getOption("-c");
+    	assertNotNull(c);
+    	assertTrue(c.isRequired());
+    }
 
 }