RFR: 8372787: ModuleReader should throw IOException consistently when using --patch-module and ModuleReader is closed [v2]

Jaikiran Pai jpai at openjdk.org
Mon Dec 1 10:56:05 UTC 2025


On Mon, 1 Dec 2025 10:38:48 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - 8372787: ModuleReader should throw IOException consistently when using --patch-module and ModuleReader is closed
>>  - Revert "8372787: ModulePatcher throws unspecified IllegalStateException upon being invoked after close()"
>>    
>>    This reverts commit 42cfa0aa40926b748c6d7bdf814d85ca1ccd8fed.
>
> src/java.base/share/classes/jdk/internal/module/ModulePatcher.java line 420:
> 
>> 418:         }
>> 419: 
>> 420:         private void ensureOpen() throws IOException {
> 
> Can you move this to after the delegate() method?

Done, I've updated the PR to move this just after the delegate() method.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28569#discussion_r2576587433


More information about the core-libs-dev mailing list