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

Unified Diff: tests/compiler/dart2js/compiler_helper.dart

Issue 11194025: Second round of cleanups for new optional parameter semantics. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 2 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
Index: tests/compiler/dart2js/compiler_helper.dart
===================================================================
--- tests/compiler/dart2js/compiler_helper.dart (revision 13754)
+++ tests/compiler/dart2js/compiler_helper.dart (working copy)
@@ -17,9 +17,9 @@
#import("mock_compiler.dart");
#import("parser_helper.dart");
-String compile(String code, [String entry = 'main',
- bool enableTypeAssertions = false,
- bool minify = false]) {
+String compile(String code, {String entry: 'main',
+ bool enableTypeAssertions: false,
+ bool minify: false}) {
MockCompiler compiler =
new MockCompiler(enableTypeAssertions: enableTypeAssertions,
enableMinification: minify);
@@ -84,13 +84,13 @@
}
void compileAndMatch(String code, String entry, RegExp regexp) {
- String generated = compile(code, entry);
+ String generated = compile(code, entry: entry);
Expect.isTrue(regexp.hasMatch(generated),
'"$generated" does not match /$regexp/');
}
void compileAndDoNotMatch(String code, String entry, RegExp regexp) {
- String generated = compile(code, entry);
+ String generated = compile(code, entry: entry);
Expect.isFalse(regexp.hasMatch(generated),
'"$generated" has a match in /$regexp/');
}
@@ -109,7 +109,7 @@
void compileAndMatchFuzzyHelper(
String code, String entry, String regexp, bool shouldMatch) {
- String generated = compile(code, entry);
+ String generated = compile(code, entry: entry);
final xRe = new RegExp('\\bx\\b');
regexp = regexp.replaceAll(xRe, '(?:$anyIdentifier)');
final spaceRe = new RegExp('\\s+');

Powered by Google App Engine
This is Rietveld 408576698