RFR: 8358558: (zipfs) Reorder the listing of "accessMode" property in the ZIP file system's documentation [v2]

Jaikiran Pai jpai at openjdk.org
Wed Jun 4 15:06:08 UTC 2025


On Wed, 4 Jun 2025 09:35:25 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Daniel's suggestion - consistent use of {@code "readOnly"}
>
> src/jdk.zipfs/share/classes/module-info.java line 195:
> 
>> 193:  *       If the value is {@code true}, the ZIP file system provider creates a
>> 194:  *       new ZIP or JAR file if it does not exist. Specifying the {@code create}
>> 195:  *       property as {@code true} with the {@code accessMode} as {@code readOnly}
> 
> should we change that to `with <em>read-only</em> {@code accessMode}` or to `with the {@code accessMode} as {@code "readOnly"}`?

I see your reasoning. In the `accessMode` property description, we have used `{@code "readOnly"}`. I will go ahead and change it to `with the {@code accessMode} as {@code "readOnly"}`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25634#discussion_r2126822336


More information about the core-libs-dev mailing list