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