On Wed, 23 Oct 2024 22:14:13 GMT, Jiangli Zhou <jian...@openjdk.org> wrote:
> Please review the fix that uses String type for the mapped value in > ModuleLoaderMap.Mapper map (Map<String, String>). Please see details in > https://bugs.openjdk.org/browse/JDK-8342642, thanks. I brought up this issue in today's Leyden/premain meeting. There was a good discussion on the specific issue, and the general Integer cache archiving and identity hash problem. The conclusion was it's better to avoid using boxed Integer and go with String in ModuleLoaderMap.Mapper. ------------- PR Comment: https://git.openjdk.org/jdk/pull/21672#issuecomment-2436262114
