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.

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

Analyzer does not handle @proxy and noSuchMethod correctly #16132

Closed
johnniwinther opened this issue Jan 16, 2014 · 9 comments
Closed

Analyzer does not handle @proxy and noSuchMethod correctly #16132

johnniwinther opened this issue Jan 16, 2014 · 9 comments
Assignees
@johnniwinther
Labels
closed-obsolete Closed as the reported issue is no longer relevant P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js

Comments

@johnniwinther
Copy link
Member

Analyzer does not handle @­proxy and noSuchMethod correctly.
@johnniwinther
Copy link
Member Author

Affected language tests:
override_inheritance_no_such_method_test/02
override_inheritance_no_such_method_test/03
override_inheritance_no_such_method_test/04
override_inheritance_no_such_method_test/05
override_inheritance_no_such_method_test/07
override_inheritance_no_such_method_test/10

Tests added in CL https://codereview.chromium.org/140803002

@bwilkerson
Copy link
Member

Added this to the 1.2 milestone.

@clayberg
Copy link

Removed Priority-Unassigned label.
Added Priority-Medium label.

@johnniwinther
Copy link
Member Author

These test were invalid and have been updated:

override_inheritance_no_such_method_test/02
override_inheritance_no_such_method_test/07
override_inheritance_no_such_method_test/10

@clayberg
Copy link

Removed this from the 1.2 milestone.
Added this to the 1.3 milestone.

@jwren
Copy link
Member

jwren commented Mar 4, 2014

Please remove tests and change behavior in dart2js. See Gilad's comments (in email).

Set owner to @johnniwinther.
Removed Area-Analyzer label.
Added Area-Dart2JS label.

@kasperl
Copy link

kasperl commented Jun 4, 2014

Removed this from the 1.3 milestone.
Added this to the 1.6 milestone.

@kasperl
Copy link

kasperl commented Jul 10, 2014

Removed this from the 1.6 milestone.
Added Oldschool-Milestone-1.6 label.

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-1.6 label.

@johnniwinther johnniwinther self-assigned this Aug 4, 2014
@kevmoo kevmoo mentioned this issue Oct 20, 2015
Closed
@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed triaged labels Mar 1, 2016
@matanlurey matanlurey added the closed-obsolete Closed as the reported issue is no longer relevant label Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnniwinther johnniwinther

Labels
closed-obsolete Closed as the reported issue is no longer relevant P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@kevmoo @matanlurey @jwren @kasperl @bwilkerson @clayberg @johnniwinther