On Fri, 12 Jun 2026 15:07:36 GMT, Alan Bateman <[email protected]> wrote:
> The changes to the module system to allow the JDK exploded image support > preview features need a new cleanups. All relatively simple: > > - the factory method to create ModuleFinder that locates modules in a JDK > exploded image is renamed as this is not like the module path specified to > the java launcher or jlink tool. > - the ModuleReader for the exploded image is checking that a located resource > exists twice. > - the ModuleReader::list method does not need to collect all resources when > preview features are disabled or the module does not have any preview classes > - Some cleanups to the test. assertUtf8Resource is renamed to testReader as > it's really testing all the module reader methods. Fixed some resource leaks > and reverted a few random format/comment changes that are nothing to do with > the changes in the branch. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: 6663aa34 Author: Alan Bateman <[email protected]> Committer: Chen Liang <[email protected]> URL: https://git.openjdk.org/valhalla/commit/6663aa34db2380a66288c74925cdf50609a9617f Stats: 160 lines in 4 files changed: 68 ins; 30 del; 62 mod 8386623: [lworld] Cleanup module support for system modules in JDK exploded build with preview resources Reviewed-by: liach ------------- PR: https://git.openjdk.org/valhalla/pull/2540
