Skip to content

Fix iterator to enhanced for loop to properly remove Iterator import #2757

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
Merged
jjohnstn merged 2 commits into from
Jan 24, 2026
Merged

Fix iterator to enhanced for loop to properly remove Iterator import #2757

jjohnstn merged 2 commits into from
Jan 24, 2026

Conversation

@jjohnstn
Copy link
Contributor

@jjohnstn jjohnstn commented Jan 23, 2026

What it does

See issue.

How to test

See issue or new test.

Author checklist

@jjohnstn
Fix iterator to enhanced for loop to properly remove Iterator import
d5040a9 
- modify UseIteratorToForLoopCleanUpCore to request a frest AST
- add new test to CleanUpTest1d8
- fixes eclipse-jdt#121
@jjohnstn jjohnstn added this to the 4.39 M3 milestone Jan 23, 2026
@jjohnstn jjohnstn self-assigned this Jan 23, 2026
@jjohnstn jjohnstn added the bug Something isn't working label Jan 23, 2026
@jjohnstn jjohnstn merged commit 7c82677 into eclipse-jdt:master Jan 24, 2026
13 checks passed
@jjohnstn jjohnstn deleted the forwhilebug branch January 26, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jjohnstn jjohnstn

Labels

bug Something isn't working

Projects

None yet

Milestone

4.39 M3

Development

Successfully merging this pull request may close these issues.

Clean up "Convert to enhanced 'for' loops" does not detect unused imports for combined enhanced for- and while-loops

1 participant

@jjohnstn