RFR: 8282429: StringBuilder/StringBuffer.toString() skip compressing for UTF16 strings [v4]
Xin Liu
xliu at openjdk.java.net
Thu Mar 24 17:54:46 UTC 2022
On Tue, 22 Mar 2022 10:09:40 GMT, Claes Redestad <redestad at openjdk.org> wrote:
>> Xin Liu has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>>
>> - Merge branch 'master' into JDK-8282429
>> - split StringBuilder.toString() performance test out of StringBuilders.java
>> - Change growOnly to maybeLatin.
>>
>> This patch also copys over the attribute from the other AbstractStringBuilder.
>> Add a unit test to cover methods which cause maybeLatin1 becomes true.
>> - make sure String(StringBuffer) is still synchronized.
>> - Add a microbenchmark.
>> - 8282429: StringBuilder/StringBuffer.toString() skip compressing for UTF16 strings
>
> Thanks for refactoring the micro to avoid the redundant run overheads, and checking that the we're size neutral on all configurations.
>
> I have added a few minor comments inline that you can choose to address or ignore.
>
> Good work!
hi, @cl4es @djelinski ,
Could you take another look at the latest revision?
thanks,
--lx
-------------
PR: https://git.openjdk.java.net/jdk/pull/7671
More information about the core-libs-dev
mailing list