Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(339)

Unified Diff: lib/args/args.dart

Issue 10855084: Added support for multi-valued options (options that can occur more than once). (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tests/lib/args/args_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/args/args.dart
===================================================================
--- lib/args/args.dart (revision 10395)
+++ lib/args/args.dart (working copy)
@@ -123,6 +123,23 @@
*
* -vfi
*
+ * By default, an option has only a single value, with later option values
+ * overriding earlier ones; for example:
+ *
+ * var parser = new ArgParser();
+ * parser.addOption('mode');
+ * var results = parser.parse(['--mode', 'on', '--mode', 'off']);
+ * print(results['mode']); // prints 'off'
+ *
+ * If you need multiple values, set the [allowMultiple] flag. In that
+ * case the option can occur multiple times and when parsing arguments a
+ * List of values will be returned:
+ *
+ * var parser = new ArgParser();
+ * parser.addOption('mode', allowMultiple: true);
+ * var results = parser.parse(['--mode', 'on', '--mode', 'off']);
+ * print(results['mode']); // prints '[on, off]'
+ *
* ## Usage ##
*
* This library can also be used to automatically generate nice usage help
@@ -214,14 +231,15 @@
*/
void addOption(String name, [String abbr, String help, List<String> allowed,
Map<String, String> allowedHelp, String defaultsTo,
- void callback(bool value)]) {
+ void callback(bool value), bool allowMultiple = false]) {
_addOption(name, abbr, help, allowed, allowedHelp, defaultsTo,
- callback, isFlag: false);
+ callback, isFlag: false, allowMultiple: allowMultiple);
}
void _addOption(String name, String abbr, String help, List<String> allowed,
Map<String, String> allowedHelp, defaultsTo,
- void callback(bool value), [bool isFlag, bool negatable = false]) {
+ void callback(bool value), [bool isFlag, bool negatable = false,
+ bool allowMultiple = false]) {
// Make sure the name isn't in use.
if (_options.containsKey(name)) {
throw new IllegalArgumentException('Duplicate option "$name".');
@@ -242,7 +260,8 @@
}
_options[name] = new _Option(name, abbr, help, allowed, allowedHelp,
- defaultsTo, callback, isFlag: isFlag, negatable: negatable);
+ defaultsTo, callback, isFlag: isFlag, negatable: negatable,
+ allowMultiple: allowMultiple);
_optionNames.add(name);
}
@@ -257,7 +276,11 @@
// Initialize flags to their defaults.
_options.forEach((name, option) {
- results[name] = option.defaultValue;
+ if (option.allowMultiple) {
+ results[name] = [];
+ } else {
+ results[name] = option.defaultValue;
+ }
});
// Parse the args.
@@ -280,9 +303,15 @@
break;
}
- // Invoke the callbacks.
+ // Set unspecified multivalued arguments to their default value,
+ // if any, and invoke the callbacks.
for (var name in _optionNames) {
var option = _options[name];
+ if (option.allowMultiple &&
+ results[name].length == 0 &&
+ option.defaultValue != null) {
+ results[name].add(option.defaultValue);
+ }
if (option.callback != null) option.callback(results[name]);
}
@@ -315,7 +344,11 @@
'"$value" is not an allowed value for option "${option.name}".');
}
- results[option.name] = value;
+ if (option.allowMultiple) {
+ results[option.name].add(value);
+ } else {
+ results[option.name] = value;
+ }
}
/**
@@ -497,10 +530,11 @@
final Map<String, String> allowedHelp;
final bool isFlag;
final bool negatable;
+ final bool allowMultiple;
_Option(this.name, this.abbreviation, this.help, this.allowed,
this.allowedHelp, this.defaultValue, this.callback, [this.isFlag,
- this.negatable]);
+ this.negatable, this.allowMultiple = false]);
}
/**
« no previous file with comments | « no previous file | tests/lib/args/args_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698