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

Unified Diff: tools/SkFlags.cpp

Issue 12961003: Allow more options for setting boolean flag values in SkFlags. (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Created 7 years, 9 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
« tools/SkFlags.h ('K') | « tools/SkFlags.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/SkFlags.cpp
diff --git a/tools/SkFlags.cpp b/tools/SkFlags.cpp
index e386b429fe5ca5dd5e2bb6e38d9401a3c841d6f2..1de597a622ad538be9c07a6fbdd2f293abac92a9 100644
--- a/tools/SkFlags.cpp
+++ b/tools/SkFlags.cpp
@@ -93,7 +93,15 @@ void SkFlags::ParseCommandLine(int argc, char** argv) {
flagMatched = true;
switch (flag->getFlagType()) {
case SkFlagInfo::kBool_FlagType:
- // Handled by match, above
+ // Can be handled by match, above, but can also be set by the next
+ // string.
+ if (i+1 < argc && !SkStrStartsWith(argv[i+1], '-')) {
+ i++;
+ bool value;
+ if (SkFlagInfo::ParseBoolArg(argv[i], &value)) {
+ flag->setBool(value);
+ }
+ }
break;
case SkFlagInfo::kString_FlagType:
flag->resetStrings();
« tools/SkFlags.h ('K') | « tools/SkFlags.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698