> One optimization of Jlink SystemModulesPlugin pre-resolves the module graph 
> for modules with a main class.  It stores the name of the initial module and 
> the generated `SystemModules` class name in two arrays that can be obtained 
> from `SystemModulesMap::moduleNames` and `SystemModulesMap::classNames`.   
> The elements in the array returned by `classNames()` are supposed to 
> correspond to the elements in the array returned by `moduleNames()`.    
> However, the implementation sorts both arrays by the value of the elements. 
> 
> This fix is simple and write the correct class names and not to sort the 
> values separately.

Mandy Chung has updated the pull request incrementally with one additional 
commit since the last revision:

  review comment

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17316/files
  - new: https://git.openjdk.org/jdk/pull/17316/files/d0e55ddb..c6e98d55

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

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

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

Reply via email to