RFR: 8048209 : (s) Collections.synchronizedNavigableSet().tailSet(Object, boolean) synchronizes on wrong object

2014-07-24 Thread Mike Duigou
Hello all; This change fixes an issue with the instance returned by Collections.synchronizedNavigableSet().tailSet(Object,boolean) which synchronizes on wrong object, itself rather than the same mutex as it's source. jbsbug: https://bugs.openjdk.java.net/browse/JDK-8048209 webrev: http://cr.ope

Re: RFR: 8048209 : (s) Collections.synchronizedNavigableSet().tailSet(Object, boolean) synchronizes on wrong object

2014-07-24 Thread Paul Sandoz
On Jul 24, 2014, at 12:12 AM, Mike Duigou wrote: > Hello all; > > This change fixes an issue with the instance returned by > Collections.synchronizedNavigableSet().tailSet(Object,boolean) which > synchronizes on wrong object, itself rather than the same mutex as it's > source. > > jbsbug: htt

Re: RFR: 8048209 : (s) Collections.synchronizedNavigableSet().tailSet(Object, boolean) synchronizes on wrong object

2014-07-24 Thread Chris Hegarty
Looks good. Trivially, should you maintain the original bugId in the test? -Chris. On 24 Jul 2014, at 08:12, Mike Duigou wrote: > Hello all; > > This change fixes an issue with the instance returned by > Collections.synchronizedNavigableSet().tailSet(Object,boolean) which > synchronizes on

Re: RFR: 8048209 : (s) Collections.synchronizedNavigableSet().tailSet(Object, boolean) synchronizes on wrong object

2014-07-24 Thread Mike Duigou
On Jul 24 2014, at 07:01 , Chris Hegarty wrote: > Looks good. > > Trivially, should you maintain the original bugId in the test? The test file is actually an hg copy. I thought I would use more of the original file but did not. I will sever the link since the tests have no relation. Mike >