RFR: 8350880: (zipfs) Add support for read-only zip file systems [v3]

David Beaumont duke at openjdk.org
Thu May 15 17:26:53 UTC 2025


On Tue, 13 May 2025 13:48:32 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> David Beaumont has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix comment based on current behaviour.
>
> src/jdk.zipfs/share/classes/jdk/nio/zipfs/ZipFileSystem.java line 174:
> 
>> 172:                 }
>> 173:                 default -> {
>> 174:                 }
> 
> Since we don't allow for any other value, I think moving the `throw new IllegalArgumentException` from outside of the switch into this `default` case might be suitable.

I rewrote to avoid the switch now there's no way it needs to cope with being the enum value.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25178#discussion_r2091660167


More information about the core-libs-dev mailing list