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
Poor readability of closures in generated JavaScript code #4003
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-enhancement
A request for a change that isn't a bug
web-dart2js
Comments
Added this to the Later milestone. |
Removed this from the Later milestone. |
Added this to the Later milestone. |
Added TriageForM5 label. |
Removed TriageForM5 label. |
Removed this from the Later milestone. |
Removed Oldschool-Milestone-Later label. |
@rakudrama Could you check what is still a problem in dart2js and update/close the bug? |
I believe it still is an issue:
|
kevmoo
added
P2
A bug or feature request we're likely to work on
type-enhancement
A request for a change that isn't a bug
and removed
triaged
labels
Feb 29, 2016
matanlurey
added
the
closed-obsolete
Closed as the reported issue is no longer relevant
label
Jun 19, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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-enhancement
A request for a change that isn't a bug
web-dart2js
There are several factors that make code containing closures essentially incomprehensible:
$$.Closure = {"":
["this_0"],
super: "Closure176",
$call$0: function() {
return this.this_0.onLoad$0();
}
};
...
$$.Closure176 = {"":
[],
super: "Object",
toString$0: function() {
return 'Closure';
}
};
I am hoping that real JavaScript functions can be used as closures since then the code could compile to the obvious JavaScript program, using JavaScript's closures. But if that is not possible, the above will still need to be addressed.
The text was updated successfully, but these errors were encountered: