On Mon, 17 Oct 2022 19:42:14 GMT, liach <d...@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

Reply via email to