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

Issue 10917006: More uses of the new try-catch syntax. (Closed)

Created:
8 years, 3 months ago by kasperl
Modified:
8 years, 3 months ago
Reviewers:
ngeoffray
CC:
reviews_dartlang.org
Visibility:
Public.

Description

More uses of the new try-catch syntax. With this change, the VM can run all tests (except co19) without allowing the use of the old try-catch syntax. R=ngeoffray@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=11600

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -33 lines) Patch
M pkg/dartdoc/dartdoc.dart View 3 chunks +3 lines, -3 lines 0 comments Download
M pkg/fixnum/test/int_32_test.dart View 2 chunks +8 lines, -8 lines 0 comments Download
M pkg/fixnum/test/int_64_test.dart View 4 chunks +11 lines, -11 lines 0 comments Download
M pkg/fixnum/test/int_64_vm_test.dart View 3 chunks +4 lines, -4 lines 0 comments Download
M pkg/intl/lib/date_format_field.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/unittest/core_matchers.dart View 3 chunks +3 lines, -3 lines 0 comments Download
M pkg/unittest/vm_config.dart View 1 chunk +1 line, -1 line 0 comments Download
M runtime/vm/exceptions_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M runtime/vm/isolate_test.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
kasperl
Another one for you, Nicolas!
8 years, 3 months ago (2012-08-30 11:04:41 UTC) #1
ngeoffray
8 years, 3 months ago (2012-08-30 11:08:28 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698