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

Jaikiran Pai jpai at openjdk.org
Mon Dec 1 16:08:35 UTC 2025


> Can I please get a review of this change which proposes to address the issue noted in https://bugs.openjdk.org/browse/JDK-8372787?
> 
> The commit in this PR updates several methods in the `jdk.internal.module.ModulePatcher$PatchedModuleReader` to throw an `IOException` if the `ModuleReader` has been closed. This updated implementation now matches the specification of the corresponding methods in `ModuleReader`.
> 
> A new jtreg test has been introduced to reproduce the issue and verify the fix. CI testing is currently in progress with this change.

Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:

  rename test methods

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28569/files
  - new: https://git.openjdk.org/jdk/pull/28569/files/30ce204c..adb01a65

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28569&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28569&range=03-04

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/28569.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28569/head:pull/28569

PR: https://git.openjdk.org/jdk/pull/28569


More information about the core-libs-dev mailing list