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

Side by Side Diff: tests/language/regress_23650_test.dart

Issue 1188103002: Do not mark type variables as needing a box inside try; they are immutable. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 6 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 unified diff | Download patch
« no previous file with comments | « pkg/compiler/lib/src/ssa/builder.dart ('k') | tests/lib/lib.status » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 import "dart:async"; 5 // Test that type variables in try-catch work.
6
6 import "package:expect/expect.dart"; 7 import "package:expect/expect.dart";
7 8
8 main() async { 9 class C<T> {
9 var error = "no error"; 10 C.foo();
10 try { 11 factory C() {
11 try { 12 try {
12 await new Future.error("error"); 13 return new C<T>.foo();
13 } on MissingType catch(e) { 14 } finally {
14 } 15 }
15 } catch (e) {
16 error = e;
17 } 16 }
18 Expect.isTrue(error is TypeError);
19 } 17 }
18
19 main() {
20 var c = new C<int>();
21 Expect.isTrue(c is C<int>);
22 Expect.isFalse(c is C<String>);
23 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/ssa/builder.dart ('k') | tests/lib/lib.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698