RFR: 8293630: Simplify TreeMap.get() with Comparator.naturalOrder() [v4]
Сергей Цыпанов
duke at openjdk.org
Fri Oct 21 12:00:53 UTC 2022
On Mon, 17 Oct 2022 19:42:14 GMT, liach <duke at openjdk.org> wrote:
>> Сергей Цыпанов has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8293630: Inline natural()
>
> src/java.base/share/classes/java/util/TreeMap.java line 3329:
>
>> 3327: }
>> 3328: else {
>> 3329: return (Comparator<Map.Entry<K, V>> & Serializable) (e1, e2) -> {
>
> I think this cast hints compiler to bootstrap lambda meta factory with extra Serializable marker interface. Meanwhile, Comparator.naturalOrder does not implement Serializable.
>From one point this is obviously a change in behaviour, from another the spec of `Spliterator.getComparator()` says nothing about serializability of returned comparator and the actual value is hardly ever going to be serialized.
-------------
PR: https://git.openjdk.org/jdk/pull/9901
More information about the core-libs-dev
mailing list