You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/05/05 02:31:55 UTC

svn commit: r535425 - /jakarta/jmeter/branches/rel-2-2/src/jorphan/org/apache/commons/cli/avalon/CLUtil.java

Author: sebb
Date: Fri May  4 17:31:54 2007
New Revision: 535425

URL: http://svn.apache.org/viewvc?view=rev&rev=535425
Log:
Fixup description for optional parameters

Modified:
    jakarta/jmeter/branches/rel-2-2/src/jorphan/org/apache/commons/cli/avalon/CLUtil.java

Modified: jakarta/jmeter/branches/rel-2-2/src/jorphan/org/apache/commons/cli/avalon/CLUtil.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/jorphan/org/apache/commons/cli/avalon/CLUtil.java?view=diff&rev=535425&r1=535424&r2=535425
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/jorphan/org/apache/commons/cli/avalon/CLUtil.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/jorphan/org/apache/commons/cli/avalon/CLUtil.java Fri May  4 17:31:54 2007
@@ -24,13 +24,19 @@
  * CLUtil offers basic utility operations for use both internal and external to
  * package.
  * 
- * @version $Revision$ $Date$
  * @see CLOptionDescriptor
  */
 public final class CLUtil {
 	private static final int MAX_DESCRIPTION_COLUMN_LENGTH = 60;
 
 	/**
+	 * Private Constructor so that no instance can ever be created.
+	 * 
+	 */
+	private CLUtil() {
+	}
+
+	/**
 	 * Format options into StringBuffer and return. This is typically used to
 	 * print "Usage" text in response to a "--help" or invalid option.
 	 * 
@@ -47,6 +53,7 @@
 			final String name = options[i].getName();
 			String description = options[i].getDescription();
 			int flags = options[i].getFlags();
+			boolean argumentOptional = ((flags & CLOptionDescriptor.ARGUMENT_OPTIONAL) == CLOptionDescriptor.ARGUMENT_OPTIONAL);
 			boolean argumentRequired = ((flags & CLOptionDescriptor.ARGUMENT_REQUIRED) == CLOptionDescriptor.ARGUMENT_REQUIRED);
 			boolean twoArgumentsRequired = ((flags & CLOptionDescriptor.ARGUMENTS_REQUIRED_2) == CLOptionDescriptor.ARGUMENTS_REQUIRED_2);
 			boolean needComma = false;
@@ -71,6 +78,9 @@
 				sb.append(name);
 			}
 
+			if (argumentOptional) {
+				sb.append(" [<argument>]");
+			}
 			if (argumentRequired) {
 				sb.append(" <argument>");
 			}
@@ -94,12 +104,5 @@
 			}
 		}
 		return sb;
-	}
-
-	/**
-	 * Private Constructor so that no instance can ever be created.
-	 * 
-	 */
-	private CLUtil() {
 	}
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org