Re: RFR: 8163353: NPE in ConcurrentHashMap.removeAll()

2016-08-24 Thread Martin Buchholz
Thanks all! Pushed. On Wed, Aug 24, 2016 at 1:06 PM, Paul Sandoz wrote: > > > On 24 Aug 2016, at 12:37, Martin Buchholz wrote: > > > > Let's get in the fix for regression 8163353 quickly: > > http://cr.openjdk.java.net/~martin/webrevs/openjdk9/ > jsr166-jdk9-integration/ConcurrentHashMap-keySe

Re: RFR: 8163353: NPE in ConcurrentHashMap.removeAll()

2016-08-24 Thread Paul Sandoz
> On 24 Aug 2016, at 12:37, Martin Buchholz wrote: > > Let's get in the fix for regression 8163353 quickly: > http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/ConcurrentHashMap-keySet-removeAll/ > https://bugs.openjdk.java.net/browse/JDK-8163353 +1 Paul.

Re: RFR: 8163353: NPE in ConcurrentHashMap.removeAll()

2016-08-24 Thread Alan Bateman
On 24/08/2016 20:37, Martin Buchholz wrote: Let's get in the fix for regression 8163353 quickly: http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/ConcurrentHashMap-keySet-removeAll/ https://bugs.openjdk.java.net/browse/JDK-8163353 Looks good, I think a couple of people

Re: RFR: 8163353: NPE in ConcurrentHashMap.removeAll()

2016-08-24 Thread Claes Redestad
+1 /Claes On 2016-08-24 21:37, Martin Buchholz wrote: Let's get in the fix for regression 8163353 quickly: http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/ConcurrentHashMap-keySet-removeAll/ https://bugs.openjdk.java.net/browse/JDK-8163353

RFR: 8163353: NPE in ConcurrentHashMap.removeAll()

2016-08-24 Thread Martin Buchholz
Let's get in the fix for regression 8163353 quickly: http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/ConcurrentHashMap-keySet-removeAll/ https://bugs.openjdk.java.net/browse/JDK-8163353