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

Analyzer: no static type warning when multiple implemented interfaces have conflicting members #7283

Closed
DartBot opened this issue Dec 11, 2012 · 6 comments
Assignees
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion.
Milestone

Comments

@DartBot
Copy link

DartBot commented Dec 11, 2012

This issue was originally filed by [email protected]


This is a duplicate of issue #3306 which was wrongfully closed

According to spec (8.1.1):
"if there are multiple members m1, …, mk with the same name n that would be inherited (because identically named members existed in several superinterfaces) then at most one member is inherited. If the static types T1, …, Tk of the members m1, …, mk are not identical, then there must be a member mx such that Tx <: Ti, 1 <= x <= k for all i, 1 <= i <= k, or a static type warning occurs."
the following should result in a static type warning:

abstract class SI1 {
  int foo;
}

abstract class SI2 {
  String foo;
}

abstract class I implements SI1, SI2 {}

However, there's currently none

co19 tests expecting this warning:
Language/08_Interfaces/4_Superinterfaces/1_Inheritance_and_Overriding_A02_t01-t04

@iposva-google
Copy link
Contributor

Added Area-Analyzer, Triaged labels.

@bwilkerson
Copy link
Member

Added this to the M3 milestone.

@bwilkerson
Copy link
Member

Removed this from the M3 milestone.
Added this to the M4 milestone.

@bwilkerson
Copy link
Member

Removed this from the M4 milestone.
Added this to the M5 milestone.

@bwilkerson
Copy link
Member

Set owner to @jwren.

@jwren
Copy link
Member

jwren commented May 31, 2013

Fixed and confirmed in svn version 23492


Added Fixed label.

@DartBot DartBot added Type-Defect area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. labels May 31, 2013
@DartBot DartBot added this to the M5 milestone May 31, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion.
Projects
None yet
Development

No branches or pull requests

4 participants