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

StringBuffer -> StringBuilder #5246

Merged
merged 2 commits into from
Jul 1, 2023

Conversation

pizzi80
Copy link
Contributor

@pizzi80 pizzi80 commented Jun 19, 2023

  • StringBuffer -> StringBuilder
  • fixed: static trimRight(String s) function inside TextUnit didn't trim anything

can I move this and other String util functions inside the Utils class?

for example inside the MessageFactory classes there is a function
that should format a String with the passed parameters and it's copy/pasted
all over the place

also it returns null if the locale parameter is null... is it expected?

@pizzi80
Copy link
Contributor Author

pizzi80 commented Jun 25, 2023

fixed 👍

@BalusC BalusC merged commit 4e766ed into eclipse-ee4j:master Jul 1, 2023
1 check passed
@arjantijms arjantijms added the 5.0 label Jul 6, 2023
@pizzi80 pizzi80 deleted the 4.1.0_stringbuilder branch July 12, 2023 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants