<i18n dev> RFR: 8332826: Make hashCode methods in ArraysSupport friendlier [v2]

Claes Redestad redestad at openjdk.org
Wed May 29 08:39:01 UTC 2024


On Wed, 29 May 2024 03:16:02 GMT, Chen Liang <liach at openjdk.org> wrote:

>> In fact, if I change it to `2`, the following tests will fail:
>> 
>>   - `jdk/jdk/classfile/Utf8EntryTest.java`
>>   - `jdk/java/util/zip/ZipCoding.java`
>>   - `jdk/java/text/Format/MessageFormat/MessageRegression.java`
>
> Indeed, the actual length passed at call site is `value.length >> 1` instead of `value.length`; this adjusted char-length carries on to `vectorizedHashCode` call.

Ah, sneaky. Might affect scores for the zero and one-char cases since the shift now happens unconditionally, but probably doesn't matter in practice.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19414#discussion_r1618456668


More information about the i18n-dev mailing list