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 takes into account the `IllegalStateException` thrown by 
`JarFile` APIs and wraps them into a `IOException` to conform with the 
expectations of the `ModuleReader` APIs.

A new jtreg test has been introduced to reproduce the issue and verify the fix. 
CI testing is currently in progress with this change.

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

Commit messages:
 - 8372787: ModulePatcher throws unspecified IllegalStateException upon being 
invoked after close()

Changes: https://git.openjdk.org/jdk/pull/28569/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=28569&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8372787
  Stats: 127 lines in 2 files changed: 124 ins; 0 del; 3 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

Reply via email to