RFR: 8186958: Need method to create pre-sized HashMap [v16]
Stuart Marks
smarks at openjdk.java.net
Wed Apr 13 23:51:16 UTC 2022
On Wed, 13 Apr 2022 20:06:34 GMT, Naoto Sato <naoto at openjdk.org> wrote:
>> XenoAmess has updated the pull request incrementally with one additional commit since the last revision:
>>
>> revert changes in:
>> src/java.desktop
>> src/java.management
>> src/jdk.internal.vm.ci
>> src/jdk.jfr
>> src/jdk.management.jfr
>> src/jdk.management
>> src/utils/IdealGraphVisualizer
>
> Looks good for changes in i18n related call sites, assuming that the copyright years will be updated.
>
> Should we need some additions/modifications to the hashmap optimal capacity utility `test/lib/jdk/test/lib/util/OptimalCapacity.java`?
@naotoj wrote:
> Should we need some additions/modifications to the hashmap optimal capacity utility `test/lib/jdk/test/lib/util/OptimalCapacity.java`?
The only thing that uses this utility now is `test/jdk/java/lang/Enum/ConstantDirectoryOptimalCapacity.java`, which is on the problem list. The cleanup is covered by [JDK-8282120](https://bugs.openjdk.java.net/browse/JDK-8282120). After this PR gets integrated, the Enum ConstantDirectory will be initialized with `HashMap.newHashMap(universe.length)` so the OptimalCapacity test won't really be testing anything. I'll take another look at it and the library utility, but I suspect the cleanup may simply be removing them entirely.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7928
More information about the security-dev
mailing list