RFR 8078645: removeIf(filter) in ConcurrentHashMap removes entries for which filter is false
David Holmes
david.holmes at oracle.com
Tue May 5 01:40:53 UTC 2015
Hi Paul,
On 5/05/2015 1:59 AM, Paul Sandoz wrote:
> Hi
>
> Please review:
>
> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8078645-ConcurrentMap-views-removeIf/webrev/
>
> These updates are already in the 166 repo.
>
> The documentation update to the class of ConcurrentMap is going through CCC. I took the liberty of also cleaning up some smaller doc errors on ConcurrentMap as made in the 166 repo.
Functional changes look okay - I'll not comment on test.
One query in ConcurrentSkipListMap, we have:
2500 // else use iterator
2501 @SuppressWarnings("unchecked")
Iterator<Map.Entry<Object,E>> it =
2502 ((SubMap<Object,E>)m).entryIterator();
and then
2578 // else use iterator
2579 Iterator<Map.Entry<K1,V1>> it =
((SubMap<K1,V1>)m).entryIterator();
why does only the former require the "unchecked" warning suppression?
Thanks,
David
> Thanks,
> Paul.
>
More information about the core-libs-dev
mailing list