Skip to content
Permalink
Browse files
[CLI-241] Clarified javadoc of OptionValidator. Thanks to Beluga Behr.
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@1544819 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
netomi committed Nov 23, 2013
1 parent ce5ebdf commit b1ffe271a5f8a6ff7b6bf714a9b2631e7dc6bbb0
Showing 3 changed files with 9 additions and 3 deletions.
@@ -36,6 +36,8 @@ NEW FEATURES:

BUG FIXES:

* Clarified behavior of "OptionValidator#validateOption(String)" in case of null input. Thanks to Beluga Behr. (CLI-241)

* Default options will now work correctly with required options that are missing. (CLI-202)

* Default options will now work correctly together with option groups. (CLI-203)
@@ -23,6 +23,9 @@
<body>

<release version="1.3" date="in SVN" description="This is a maintenance release containing bug fixes.">
<action type="fix" dev="tn" issue="CLI-241" due-to="Beluga Behr">
Clarified behavior of "OptionValidator#validateOption(String)" in case of null input.
</action>
<action type="update" dev="tn" issue="CLI-240" due-to="Beluga Behr">
Small cleanup of Option class.
</action>
@@ -31,19 +31,20 @@
* is valid are:
*
* <ul>
* <li><code>opt</code> is not NULL</li>
* <li>a single character <code>opt</code> that is either
* ' '(special case), '?', '@' or a letter</li>
* <li>a multi character <code>opt</code> that only contains
* letters.</li>
* </ul>
* <p>
* In case {@code opt} is {@code null} no further validation is performed.
*
* @param opt The option string to validate
* @param opt The option string to validate, may be null
* @throws IllegalArgumentException if the Option is not valid.
*/
static void validateOption(String opt) throws IllegalArgumentException
{
// check that opt is not NULL
// if opt is NULL do not check further
if (opt == null)
{
return;

0 comments on commit b1ffe27

Please sign in to comment.