> Summary of changes:
> 
> Before dumping info the archive, all the module names from `--add-modules` 
> will be sorted and then concatenated into one string with comma as the 
> separator between module names.
> 
> During runtime, same function will be used to obtain the string in the same 
> format with sorted module names. The string will be compared with the one 
> from the archive to determine if the same module names were specified during 
> dump time.
> 
> ModuleBootstrap.java
>         The `addModulesFromRuntimeImage` method is added to check if the 
> modules in `addModules` are from the runtime image. If any of the modules 
> isn't in the runtime image, archiving will be disabled.
> 
> ArchivedModuleGraph.java
>        The constructor has an addition argument `Set addModules`
>        The `get` method also has an addition argument `Set addModules`. It 
> returns the `archivedModuleGraph` only if both the `mainModule` and the 
> `addModules` are the same as the input arguments.
> 
> Passed tiers 1 - 4 testing.

Calvin Cheung has updated the pull request incrementally with one additional 
commit since the last revision:

  @AlanBateman comment

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21553/files
  - new: https://git.openjdk.org/jdk/pull/21553/files/5c2783f7..3d46102a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21553&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21553&range=01-02

  Stats: 41 lines in 1 file changed: 12 ins; 28 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/21553.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21553/head:pull/21553

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

Reply via email to