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

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

Issue 10987073: Fix issue 5517 by setting the successors the right way in a try/catch. (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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « lib/compiler/implementation/ssa/validate.dart ('k') | tests/language/language_dart2js.status » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
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.
4
5 // Test for a bug in dart2js where the update of a field in a try
6 // block would not be seen by the finally block. See dartbug.com/5517.
7
8 class A {
9 int i;
10 A() : i = 42;
11
12 foo() {
13 bool executedFinally = false;
14 if (i === 42) {
15 try {
16 i = 12;
17 } finally {
18 Expect.equals(12, i);
19 executedFinally = true;
20 }
21 }
22 Expect.isTrue(executedFinally);
23 }
24 }
25
26 main() {
27 new A().foo();
28 }
OLDNEW
« no previous file with comments | « lib/compiler/implementation/ssa/validate.dart ('k') | tests/language/language_dart2js.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698