On Tue, 9 Jan 2024 18:13:43 GMT, Mandy Chung <mch...@openjdk.org> wrote:
>> 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 Marked as reviewed by alanb (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/17316#pullrequestreview-1811743456