RFR: 8347712: IllegalStateException on multithreaded ZipFile access with non-UTF8 charset [v8]

Jaikiran Pai jpai at openjdk.org
Mon May 5 10:34:48 UTC 2025


On Fri, 2 May 2025 19:06:58 GMT, Shaojin Wen <swen at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Lance's review - update code comment in the test
>
> src/java.base/share/classes/java/util/zip/ZipFile.java line 1240:
> 
>> 1238:                 entries[index] = hash;
>> 1239:                 entries[index + 1] = next;
>> 1240:                 entries[index + 2] = pos;
> 
> Suggestion:
> 
>                 entries[index    ] = hash;
>                 entries[index + 1] = next;
>                 entries[index + 2] = pos;
> 
> Aligned code is more readable

Hello Shaojin, leaving one or more spaces before a closing `]` bracket of array access contradicts the style that I've seen used often in the code. At least it's not a common style used in this area of the code in the JDK.

I think it's fine in the current form and changing it to align with subsequent lines isn't necessary.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23986#discussion_r2073207272


More information about the core-libs-dev mailing list