Skip to content

Commit 0564458

Browse files
committed
lightweight notation when iterating over OptionGroup.getOptions() elements
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@1095600 13f79535-47bb-0310-9956-ffa450edef68
1 parent 8f77d1d commit 0564458

1 file changed

Lines changed: 1 addition & 6 deletions

File tree

src/main/java/org/apache/commons/cli/Options.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
import java.util.Collections;
2424
import java.util.HashMap;
2525
import java.util.HashSet;
26-
import java.util.Iterator;
2726
import java.util.LinkedHashMap;
2827
import java.util.List;
2928
import java.util.Map;
@@ -69,17 +68,13 @@ public class Options implements Serializable
6968
*/
7069
public Options addOptionGroup(OptionGroup group)
7170
{
72-
Iterator<Option> options = group.getOptions().iterator();
73-
7471
if (group.isRequired())
7572
{
7673
requiredOpts.add(group);
7774
}
7875

79-
while (options.hasNext())
76+
for (Option option : group.getOptions())
8077
{
81-
Option option = options.next();
82-
8378
// an Option cannot be required if it is in an
8479
// OptionGroup, either the group is required or
8580
// nothing is required

0 commit comments

Comments
 (0)