geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jdil...@apache.org
Subject svn commit: r593400 - /geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/CommandLineProcessor.java
Date Fri, 09 Nov 2007 02:36:53 GMT
Author: jdillon
Date: Thu Nov  8 18:36:52 2007
New Revision: 593400

URL: http://svn.apache.org/viewvc?rev=593400&view=rev
Log:
(GSHELL-36) Drop FIXME comment, cause its fixed

Modified:
    geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/CommandLineProcessor.java

Modified: geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/CommandLineProcessor.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/CommandLineProcessor.java?rev=593400&r1=593399&r2=593400&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/CommandLineProcessor.java
(original)
+++ geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/CommandLineProcessor.java
Thu Nov  8 18:36:52 2007
@@ -282,11 +282,6 @@
             // Keep a list of the handlers which have been processed (for required validation
below)
             present.add(handler);
         }
-
-        //
-        // FIXME: Some options, like --help or --version may be used in conjunction with
other options which are marked as required.
-        //        We need to set a little flag on the --help|--verison options to override
this  required handling crapo...
-        //
         
         // Ensure that all required option handlers are present
         if (!overrideRequiredArguments) {



Mime
View raw message