cuter usage for ExplicitBoolean

args4j/src/org/kohsuke/args4j/spi/ExplicitBooleanOptionHandler.java

@@ -5,10 +5,12 @@
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.ResourceBundle;
 
 import org.kohsuke.args4j.CmdLineException;
 import org.kohsuke.args4j.CmdLineParser;
 import org.kohsuke.args4j.OptionDef;
+import org.kohsuke.args4j.ParserProperties;
 
 /**
  * Boolean {@link OptionHandler} that (unlike the standard {@link BooleanOptionHandler}
@@ -61,4 +63,26 @@ private Boolean getBoolean(String parameter) throws CmdLineException {
     public String getDefaultMetaVariable() {
         return Messages.DEFAULT_META_EXPLICIT_BOOLEAN_OPTION_HANDLER.format();
     }
+
+    @Override
+    public String getNameAndMeta(ResourceBundle rb, ParserProperties properties) {
+        String str = option.isArgument() ? "" : option.toString();
+        String meta = getMetaVariable(rb);
+        if (meta != null) {
+            if (option.isArgument()) {
+                str += '[';
+            } else {
+                String delimiter = properties.getOptionValueDelimiter();
+                if (delimiter.trim().equals(delimiter)) {
+                    // not blank, bracket before delimiter
+                    str += '[' + delimiter;
+                } else {
+                    // blank, bracket after space
+                    str += delimiter + '[';
+                }
+            }
+            str += meta + ']';
+        }
+        return str;
+    }
 }

args4j/src/org/kohsuke/args4j/spi/Messages.properties

@@ -25,7 +25,7 @@ ILLEGAL_UUID = \
 ILLEGAL_PATH = \
     Failed to parse path "{0}"
 DEFAULT_META_EXPLICIT_BOOLEAN_OPTION_HANDLER = \
-    VALUE
+    true|false
 DEFAULT_META_FILE_OPTION_HANDLER = \
     FILE
 DEFAULT_META_INET_ADDRESS_OPTION_HANDLER = \

args4j/src/org/kohsuke/args4j/spi/Messages_de.properties

@@ -33,7 +33,7 @@ ILLEGAL_UUID = \
 ILLEGAL_PATH = \
     Pfad "{0}" ist nicht g\u00fcltig
 DEFAULT_META_EXPLICIT_BOOLEAN_OPTION_HANDLER = \
-    WERT
+    true|false
 DEFAULT_META_FILE_OPTION_HANDLER = \
     DATEI
 DEFAULT_META_INET_ADDRESS_OPTION_HANDLER = \

args4j/src/org/kohsuke/args4j/spi/OptionHandler.java

@@ -107,7 +107,7 @@ public final String getNameAndMeta(ResourceBundle rb) {
      * @param properties
      *      Affects the formatting behaviours.
      */
-    public final String getNameAndMeta(ResourceBundle rb, ParserProperties properties) {
+    public String getNameAndMeta(ResourceBundle rb, ParserProperties properties) {
     	String str = option.isArgument() ? "" : option.toString();
     	String meta = getMetaVariable(rb);
     	if (meta != null) {

args4j/test/org/kohsuke/args4j/ExplicitBooleanArgumentTest.java

@@ -94,7 +94,8 @@ public void testUsage() {
         try {
             parser.parseArgument(args);
         } catch (CmdLineException e) {
-            assertUsageContains("Usage message should contain 'VAL'", "VAL");
+            // This tests 2 things
+            assertUsageContains("Usage message should contain '[true|false]'", "[true|false]");
         }
     }
 

args4j/test/org/kohsuke/args4j/ExplicitBooleanOptionTest.java

@@ -100,11 +100,22 @@ public void testIllegalBoolean() {
 
     public void testUsage() {
         args = new String[] { "-wrong" };
+
         try {
             parser.parseArgument(args);
         } catch (CmdLineException e) {
-            assertUsageContains("Usage message should contain 'VAL'", "VAL");
+            assertUsageContains("Usage message should contain '[true|false]'", "[true|false]");
         }
     }
 
+    public void testUsageWithEquals() {
+        args = new String[] { "-wrong" };
+        parser = new CmdLineParser(testObject, ParserProperties.defaults().withOptionValueDelimiter("="));
+
+        try {
+            parser.parseArgument(args);
+        } catch (CmdLineException e) {
+            assertUsageContains("Usage message should contain '[=true|false]'", "[=true|false]");
+        }
+    }
 }