RFR: 8350880: (zipfs) Add support for read-only zip file systems [v4]
David Beaumont
duke at openjdk.org
Mon May 19 11:35:55 UTC 2025
On Wed, 14 May 2025 16:42:07 GMT, Lance Andersen <lancea at openjdk.org> wrote:
>> David Beaumont has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Changes based on review feedback.
>
> test/jdk/jdk/nio/zipfs/TestPosix.java line 434:
>
>> 432: createTestZipFile(ZIP_FILE, ENV_DEFAULT).close();
>> 433: // check entries on zipfs with default options
>> 434: try (FileSystem zip = FileSystems.newFileSystem(ZIP_FILE, ENV_DEFAULT)) {
>
> This tests an empty Map and should still be a test as it is different from ENV_READ_ONLY
Revert and added new ReadOnly variants of those tests. Thanks for calling me out on this, it was a bit sloppy of me.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25178#discussion_r2095503763
More information about the core-libs-dev
mailing list