You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2015/05/03 19:09:29 UTC
svn commit: r1677451 -
/commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/PosixParser.java
Author: ggregory
Date: Sun May 3 17:09:29 2015
New Revision: 1677451
URL: http://svn.apache.org/r1677451
Log:
Statement unnecessarily nested within else clause.
Modified:
commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/PosixParser.java
Modified: commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/PosixParser.java
URL: http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/PosixParser.java?rev=1677451&r1=1677450&r2=1677451&view=diff
==============================================================================
--- commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/PosixParser.java (original)
+++ commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/PosixParser.java Sun May 3 17:09:29 2015
@@ -154,11 +154,8 @@ public class PosixParser extends Parser
{
throw new AmbiguousOptionException(token, matchingOpts);
}
- else
- {
- Option opt = options.getOption(matchingOpts.get(0));
- processOptionToken("-" + opt.getLongOpt(), stopAtNonOption);
- }
+ Option opt = options.getOption(matchingOpts.get(0));
+ processOptionToken("-" + opt.getLongOpt(), stopAtNonOption);
}
// requires bursting
else