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

[dart2js] Zero-effect code not tree-shaken off #11559

Closed
yjbanov opened this issue Jun 27, 2013 · 3 comments
Closed

[dart2js] Zero-effect code not tree-shaken off #11559

yjbanov opened this issue Jun 27, 2013 · 3 comments
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. 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

@yjbanov
Copy link

yjbanov commented Jun 27, 2013

INPUT:

main() {
  new Foo(); // Has no effect
}

class Foo {}

OUTPUT (reformatted):

main(){
  new Foo(); // expected to be gone
}

class Foo{} // expected to be gone
@anders-sandholm
Copy link
Contributor

Added Area-Dart2Dart, Triaged labels.

@sethladd sethladd added the closed-not-planned Closed as we don't intend to take action on the reported issue label Jul 10, 2015
@yjbanov yjbanov changed the title [dart2dart] Zero-effect code not tree-shaken off [dart2js] Zero-effect code not tree-shaken off Jul 10, 2015
@yjbanov yjbanov added Priority-Medium web-dart2js and removed Area-Dart2Dart closed-not-planned Closed as we don't intend to take action on the reported issue labels Jul 10, 2015
@yjbanov
Copy link
Author

yjbanov commented Jul 10, 2015

This issue was misclassified. Reopening and moving to dart2js.

@yjbanov yjbanov reopened this Jul 10, 2015
@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 Feb 29, 2016
@vsmenon vsmenon added the area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. label Jul 20, 2019
@rakudrama
Copy link
Member

I can't repro anything that looks like this.

The current generated JavaScript is

    main() {
    }

with no sign of Foo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. 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
6 participants
@sethladd @kevmoo @yjbanov @rakudrama @vsmenon @anders-sandholm