On Sat, 4 Jun 2022 11:27:00 GMT, Andrey Turbanov <[email protected]> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Changed per suggestion
>
> src/java.base/share/classes/java/lang/String.java line 849:
>
>> 847: int en = scale(len, ce.maxBytesPerChar());
>> 848: // fastpath with ArrayEncoder implies `doReplace`.
>> 849: if (ce instanceof ArrayEncoder ae && doReplace) {
>
> Wouldn't it be more readable (and perphaps faster) if we do check boolean
> flag first?
Fixed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/9019
