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
Avoid allocating redundant Strings #13085
Conversation
Removed unnecessary `toString()` / `substring()` calls to avoid extra Strings being allocated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some terrible cruft in there, thank you. Please add the changes entry and credit yourself.
Done. |
This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the dev@lucene.apache.org list. Thank you for your contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I just had to move the CHANGES entry up. |
Thank you, @stefanvodita |
Removed unnecessary
toString()
/substring()
calls to avoid extra Strings being allocated