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

If class is referenced only via undefined named constructor, it is not emitted to output #6895

Closed
DartBot opened this issue Nov 23, 2012 · 10 comments
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue

Comments

@DartBot
Copy link

DartBot commented Nov 23, 2012

This issue was originally filed by smok@google.com


Example (from tests/co19/src/Language/11_Expressions/11_Instance_Creation/1_New_A03_t01.dart):

class A{}
main() {
  new A.named();
}

Result:
main(){new p_Unresolved();}

Or should 'A' get into 'instantiated classes' collection?

@DartBot
Copy link
Author

DartBot commented Nov 23, 2012

This comment was originally written by antonm@google.com


but why if you cannot instantiate it? And there is no other references?

@DartBot
Copy link
Author

DartBot commented Nov 23, 2012

This comment was originally written by smok@google.com


Why we should emit it? Different error type. Compile-time unresolved reference in case of "new Unresolved()" and NoSuchMethodError in case of "new A.unresolved();"

@DartBot
Copy link
Author

DartBot commented Nov 23, 2012

This comment was originally written by antonm@google.com


Thanks a lot for explanations. That's a crap, I must say.

@kasperl
Copy link

kasperl commented Dec 14, 2012

Added this to the M3 milestone.

@kasperl
Copy link

kasperl commented Dec 14, 2012

Feel free to take ownership again if you plan on working on this, Roman.


Removed the owner.

@anders-sandholm
Copy link
Contributor

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

@larsbak
Copy link

larsbak commented May 28, 2013

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

@kasperl
Copy link

kasperl commented May 31, 2013

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

@kasperl
Copy link

kasperl commented Jul 10, 2014

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

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-Later label.

@sethladd sethladd added the closed-not-planned Closed as we don't intend to take action on the reported issue label Jul 10, 2015
@kevmoo kevmoo removed the triaged label Mar 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue
Projects
None yet
Development

No branches or pull requests

6 participants