RFR: JDK-8310913 Move ReferencedKeyMap to jdk.internal so it may be shared [v5]
Mandy Chung
mchung at openjdk.org
Tue Jul 18 16:25:06 UTC 2023
On Tue, 18 Jul 2023 15:01:11 GMT, Chen Liang <liach at openjdk.org> wrote:
>> src/java.base/share/classes/jdk/internal/util/ReferencedKeyMap.java line 132:
>>
>>> 130: */
>>> 131: public static <K, V> ReferencedKeyMap<K, V>
>>> 132: create(boolean isSoft, boolean useNativeQueue, Supplier<Map<ReferenceKey<K>, V>> supplier) {
>>
>> I suggest to keep the previous `create(boolean isSoft, Supplier supplier)` factory method that defaults to non-native reference queue, which is commonly used. Only `MethodType` needs to use native reference queue.
>
> Mandy: The no-`NativeQueue` version is already implemented above.
Yes Jim made the change per this review feedback - see [8eff918](https://github.com/openjdk/jdk/pull/14684/commits/8eff91836f98cb33164046f2304185f11df12609)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14684#discussion_r1267020337
More information about the core-libs-dev
mailing list