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

Unified Diff: tools/testing/dart/test_options.dart

Issue 9109012: Fix static type warnings in test.dart. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 12 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 | « tools/testing/dart/multitest.dart ('k') | tools/testing/dart/test_progress.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/test_options.dart
diff --git a/tools/testing/dart/test_options.dart b/tools/testing/dart/test_options.dart
index 0d6e72104c2ba26b03e0ae059a0104ef2aabc10f..79ebb2d978e83c710d71e3ed8a6747a1d13d36a1 100644
--- a/tools/testing/dart/test_options.dart
+++ b/tools/testing/dart/test_options.dart
@@ -406,11 +406,11 @@ is 'dart file.dart' and you specify special command
var buffer = new StringBuffer();;
buffer.add(name);
if (option.type == 'bool') {
- assert(option.values.empty());
+ assert(option.values.isEmpty());
} else {
buffer.add(name.startsWith('--') ? '=' : ' ');
if (option.type == 'int') {
- assert(option.values.empty());
+ assert(option.values.isEmpty());
buffer.add('n (default: ${option.defaultValue})');
} else {
buffer.add('[');
« no previous file with comments | « tools/testing/dart/multitest.dart ('k') | tools/testing/dart/test_progress.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698