<Swing Dev> RFR: 8264428: Cleanup usages of StringBuffer in java.desktop [v3]

Alexey Ivanov aivanov at openjdk.java.net
Wed Apr 7 15:30:40 UTC 2021


On Wed, 7 Apr 2021 06:39:48 GMT, Andrey Turbanov <github.com+741251+turbanoff at openjdk.org> wrote:

>> There are few possible cleanups in java.desktop related to legacy StringBuffer usages:
>> 1. In few places StringBuffer can be replaced with plain String concatenation.
>> 2. StringBuffer can be replaced with StringBuilder. StringBuilder has better performance as it is not thread-safe.
>> 3. There are few places where result of string concatenation is passed to StringBuffer.append method. Using separate `.append` calls is more clear.
>> 4. In few places primitives are unnecessary converted to String before `.append` call. They can be replaced with specialized methods (like `.append(int)` calls.
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   [PATCH] Replace uses of StringBuffer with StringBuilder in java.desktop
>   
>   fix copyright year

Marked as reviewed by aivanov (Reviewer).

src/java.desktop/unix/classes/sun/awt/X11/XCreateWindowParams.java line 88:

> 86:         while (eIter.hasNext()) {
> 87:             Map.Entry<Object, Object> entry = eIter.next();
> 88:             buf.append(entry.getKey()).append(": ").append(entry.getValue()).append("\n");

Would it be clearer if each append was on its own line?
Suggestion:

            buf.append(entry.getKey())
               .append(": ")
               .append(entry.getValue())
               .append("\n");

-------------

PR: https://git.openjdk.java.net/jdk/pull/3251


More information about the swing-dev mailing list