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

Reduce code duplication in JDTUtils.getFieldAccessors(). #1075

Closed
mrglavas opened this issue Nov 5, 2024 · 0 comments · Fixed by #1076
Closed

Reduce code duplication in JDTUtils.getFieldAccessors(). #1075

mrglavas opened this issue Nov 5, 2024 · 0 comments · Fixed by #1076
Assignees
@mrglavas
Labels
clean-up language client language server integration
Milestone
24.0.12

Comments

@mrglavas
Copy link
Contributor

mrglavas commented Nov 5, 2024

Replace this sequence of calls (basically an unrolled loop) with a list collector.

accessorNames.add("get" + fieldName);
accessorNames.add("set" + fieldName);
accessorNames.add("is" + fieldName);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mrglavas mrglavas

Labels
clean-up language client language server integration
Projects
Open Liberty Developer Experience
Status: Sprint Closed
Milestone
24.0.12
Development

Successfully merging a pull request may close this issue.

Reduce code duplication in JDTUtils.getFieldAccessors() mrglavas/liberty-tools-intellij
1 participant
@mrglavas