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 allows mixing in classes that you are not allowed to implement or extend. #12445

Closed
lrhn opened this issue Aug 14, 2013 · 6 comments
Closed

Dart2js allows mixing in classes that you are not allowed to implement or extend. #12445

lrhn opened this issue Aug 14, 2013 · 6 comments
Assignees
@johnniwinther
Labels
web-dart2js
Milestone
M8

Comments

@lrhn
Copy link
Member

lrhn commented Aug 14, 2013

Example code:

  class C extends Object with String {}
  main() {
    print(new C() is String); // Prints true!
  }

Since the mixin application does implement its mixin class, it should be prevented from mixing in String and other classes that the user may not extend or implement.
@kasperl
Copy link

kasperl commented Sep 18, 2013

Added this to the M7 milestone.

@johnniwinther
Copy link
Member

Set owner to @johnniwinther.
Added Accepted label.

@kasperl
Copy link

kasperl commented Sep 30, 2013

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

@kasperl
Copy link

kasperl commented Oct 2, 2013

Removed this from the M7 milestone.
Added this to the M8 milestone.

@johnniwinther
Copy link
Member

Added Started label.

@johnniwinther
Copy link
Member

Fixed by https://codereview.chromium.org//25844002

Added Fixed label.

@lrhn lrhn added this to the M8 milestone Oct 3, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnniwinther johnniwinther

Labels
web-dart2js
Projects
None yet
Milestone
M8
Development

No branches or pull requests
3 participants
@kasperl @lrhn @johnniwinther