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

Frog generates non-working code for multiple closures over one mutable variable. #531

Closed
rakudrama opened this issue Nov 19, 2011 · 3 comments
Closed

Frog generates non-working code for multiple closures over one mutable variable. #531

rakudrama opened this issue Nov 19, 2011 · 3 comments
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue web-dart2js

Comments

@rakudrama
Copy link
Member

This happens only in loops (reentrant nested scopes).

See test case: http://codereview.chromium.org/8602013/
@jmesserly
Copy link

BTW, I think this happens any time you mutate something after capturing it.
Yeah, we need to be lifting binding against a scope object that we create to hold mutable captures, rather than binding to the values.

@anders-sandholm
Copy link
Contributor

Removed Area-Frog label.
Added Area-Dart2JS, FromAreaFrog labels.

@kasperl
Copy link

kasperl commented Jun 11, 2012

Added WontFix label.

@kevmoo kevmoo added closed-not-planned Closed as we don't intend to take action on the reported issue and removed resolution-wont_fix labels Mar 1, 2016
nex3 pushed a commit that referenced this issue Aug 31, 2016
Make parts hermetic, fixes #531
9f764bf 
[email protected], [email protected]

Review URL: https://codereview.chromium.org/1930133002 .
This was referenced Oct 29, 2020
Open
Closed
Closed
This was referenced Nov 5, 2020
Closed
Open
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue web-dart2js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kevmoo @jmesserly @rakudrama @kasperl @anders-sandholm