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

Range over int #1644

Closed
wants to merge 2 commits into from
Closed

Range over int #1644

wants to merge 2 commits into from

Conversation

rcoreilly
Copy link
Contributor

This adds support for the for i := range n range-over-int form of the for loop, introduced in go 1.22 golang/go#61405

Includes tests for 3 forms, and everything passes.

@rcoreilly
Copy link
Contributor Author

This is now superseded by #1645.

@rcoreilly rcoreilly closed this Jul 11, 2024
traefiker pushed a commit that referenced this pull request Jul 20, 2024
…o 1.22 behavior

This builds on #1644 and adds automatic per-loop variables that are consistent with go 1.22 behavior.  See #1643 for discussion.

This is still a draft because the for7 version ends up capturing the per-loop var values when they are +1 relative to what they should be.  Maybe somehow the incrementing and conditional code is somehow capturing the within loop variables and incrementing them?  not sure how that would work.  anyway, need to investigate further before this is ready to go, but pushing it here in case there are other issues or someone might figure out this bug before I do..
Bai-Yingjie pushed a commit to godevsig/yaegi that referenced this pull request Oct 9, 2024
…o 1.22 behavior

This builds on traefik#1644 and adds automatic per-loop variables that are consistent with go 1.22 behavior.  See traefik#1643 for discussion.

This is still a draft because the for7 version ends up capturing the per-loop var values when they are +1 relative to what they should be.  Maybe somehow the incrementing and conditional code is somehow capturing the within loop variables and incrementing them?  not sure how that would work.  anyway, need to investigate further before this is ready to go, but pushing it here in case there are other issues or someone might figure out this bug before I do..
Bai-Yingjie pushed a commit to godevsig/yaegi that referenced this pull request Oct 9, 2024
…o 1.22 behavior

This builds on traefik#1644 and adds automatic per-loop variables that are consistent with go 1.22 behavior.  See traefik#1643 for discussion.

This is still a draft because the for7 version ends up capturing the per-loop var values when they are +1 relative to what they should be.  Maybe somehow the incrementing and conditional code is somehow capturing the within loop variables and incrementing them?  not sure how that would work.  anyway, need to investigate further before this is ready to go, but pushing it here in case there are other issues or someone might figure out this bug before I do..
Bai-Yingjie pushed a commit to godevsig/yaegi that referenced this pull request Oct 9, 2024
…o 1.22 behavior

This builds on traefik#1644 and adds automatic per-loop variables that are consistent with go 1.22 behavior.  See traefik#1643 for discussion.

This is still a draft because the for7 version ends up capturing the per-loop var values when they are +1 relative to what they should be.  Maybe somehow the incrementing and conditional code is somehow capturing the within loop variables and incrementing them?  not sure how that would work.  anyway, need to investigate further before this is ready to go, but pushing it here in case there are other issues or someone might figure out this bug before I do..
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant