Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

dart2js and vm differ about whether 'var' can be omitted in catch clause #1788

Closed
DartBot opened this issue Feb 21, 2012 · 5 comments
Closed

dart2js and vm differ about whether 'var' can be omitted in catch clause #1788

DartBot opened this issue Feb 21, 2012 · 5 comments
Assignees
@peter-ahe-google
Labels
closed-as-intended Closed as the reported issue is expected behavior P1 A high priority bug; for example, a single project is unusable or has many test failures web-dart2js
Milestone
M1

Comments

@DartBot
Copy link

DartBot commented Feb 21, 2012

This issue was originally filed by mattsh@google.com

I'm trying to run this program:

void main() {
  try {
    throw new Exception("hello exception");
  } catch (e) {
    print("caught '$e'");
  }
}

This runs without error in frog. But, on the vm, it reports:

Error: line 4 pos 12: type 'e' is not loaded

(I'm not sure which behavior is what we want, so marking this a lang issue.)
@kasperl
Copy link

kasperl commented Feb 22, 2012

The VM is correct here. Marking this as a Frog bug.

Removed Area-Language label.
Added Area-Frog label.

@anders-sandholm
Copy link
Contributor

Removed Area-Frog label.
Added Area-Dart2JS, FromAreaFrog labels.

@kasperl
Copy link

kasperl commented Jun 12, 2012

This is a dart2js bug.

Set owner to @peter-ahe-google.
Removed FromAreaFrog label.
Added Accepted label.
Changed the title to: "dart2js and vm differ about whether 'var' can be omitted in catch clause".

@peter-ahe-google
Copy link
Contributor

Added this to the M1 milestone.
Removed Priority-Medium label.
Added Priority-High label.

@peter-ahe-google
Copy link
Contributor

No longer an issue, see bug 2948.

Added AsDesigned label.

@DartBot DartBot added Type-Defect P1 A high priority bug; for example, a single project is unusable or has many test failures web-dart2js closed-as-intended Closed as the reported issue is expected behavior labels Aug 23, 2012
@DartBot DartBot added this to the M1 milestone Aug 23, 2012
copybara-service bot pushed a commit that referenced this issue Nov 28, 2022
@devoncarew
[deps] rev ffi, fixnum, http, markdown, protobuf, sse, test
32ae442 
Revisions updated by `dart tools/rev_sdk_deps.dart`.

ffi (https://github.com/dart-lang/ffi/compare/3ede231..17a8142):
  17a8142  2022-11-21  Daco Harkes  Add links to API reference and examples to README (#167)

fixnum (https://github.com/dart-lang/fixnum/compare/bca3816..62916f2):
  62916f2  2022-11-24  Lasse R.H. Nielsen  Split into separate libraries instead of using parts. (#97)
  14d4827  2022-11-23  Lasse R.H. Nielsen  Add `tryParse` methods. (#96)

http (https://github.com/dart-lang/http/compare/d56141d..047d6ed):
  047d6ed  2022-11-22  Brian Quinlan  Fixes a bug where requests made in different Clients would fail (#827)
  9dbbafb  2022-11-17  Brian Quinlan  Add tests for compressed responses (#823)

markdown (https://github.com/dart-lang/markdown/compare/37951d1..ee3f4e9):
  ee3f4e9  2022-11-24  Zhiguang Chen  Fix a blockquote issue (#496)
  c9048c0  2022-11-16  Zhiguang Chen  Optimise DelimiterSyntax (#492)

protobuf (https://github.com/dart-lang/protobuf/compare/ae90e53..c181573):
  c181573  2022-11-23  Kevin Moore  [api_benchmark] migrate to null-safety (#776)
  648deaf  2022-11-23  Kevin Moore  Standardize and fix lints (#775)
  bfa4c0d  2022-11-23  Kevin Moore  Fix `///` at the top of generated Dart files (#774)
  ed68426  2022-11-17  Devon Carew  Remove duplicate consts (#773)

sse (https://github.com/dart-lang/sse/compare/283568d..8d018dd):
  8d018dd  2022-11-23  Elliott Brooks (she/her)  Format markdown files (#68)
  2f6f151  2022-11-23  Elliott Brooks (she/her)  Add optional `debugKey` parameter to SSE client (#67)
  eaee6a8  2022-11-23  Elliott Brooks (she/her)  Use Fetch API in SSE Client (#66)

test (https://github.com/dart-lang/test/compare/7756833..b25dac9):
  b25dac99  2022-11-21  Kevin Moore  checks: finish async tests (#1793)
  3166163e  2022-11-18  Devon Carew  Update scorecards-analysis.yml (#1792)
  d930d5b0  2022-11-18  Nate Bosch  Add support for async soft checks (#1789)
  b1411a21  2022-11-18  Devon Carew  blast_repo fixes (#1788)
  a6aa04e0  2022-11-18  Jacob MacDonald  disable wasm tests for now as they are failing (#1791)
  8b5174c1  2022-11-17  Kevin Moore  Starting on async tests (#1787)

Change-Id: Ic361fce7753d08d43fc827f13cb1bfa1738cc16e
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/272343
Auto-Submit: Devon Carew <devoncarew@google.com>
Reviewed-by: Kevin Moore <kevmoo@google.com>
Commit-Queue: Devon Carew <devoncarew@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peter-ahe-google peter-ahe-google

Labels
closed-as-intended Closed as the reported issue is expected behavior P1 A high priority bug; for example, a single project is unusable or has many test failures web-dart2js
Projects
None yet
Milestone
M1
Development

No branches or pull requests
4 participants
@kasperl @anders-sandholm @peter-ahe-google @DartBot