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

Unified Diff: pkg/dev_compiler/test/codegen/language/initializing_formal_scope_test.dart

Issue 2534293002: Migrate tests for initializing formals (Closed)
Patch Set: Created 4 years, 1 month 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: pkg/dev_compiler/test/codegen/language/initializing_formal_scope_test.dart
diff --git a/pkg/dev_compiler/test/codegen/language/initializing_formal_scope_test.dart b/pkg/dev_compiler/test/codegen/language/initializing_formal_scope_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..5544a8e0c42d232cbb2a7bed91faeb168c65b5aa
--- /dev/null
+++ b/pkg/dev_compiler/test/codegen/language/initializing_formal_scope_test.dart
@@ -0,0 +1,26 @@
+// Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+//
+// DartOptions=--initializing-formal-access
+// VMOptions=--initializing-formal-access
+
+import "package:expect/expect.dart";
+
+// Duplicate definition checks for `this.x` will check the scopes associated
+// with the constructor, not all enclosing scopes; so this is not a conflict.
+var x;
+
+class A {
+ var x;
+ A(this.x) {
+ // In the body the field is in scope, not the initializing formal;
+ // so we can use the setter.
+ x += 1;
+ }
+}
+
+main() {
+ A a = new A(2);
+ Expect.equals(a.x, 3);
+}

Powered by Google App Engine
This is Rietveld 408576698