jdk (original) (raw)

Code Review for jdk

Prepared by: mduigou on Thu Jul 24 00:07:26 PDT 2014
Workspace: /home/mike/code/jdk/jdk9-dev/jdk
Compare against: ssh://hg.openjdk.java.net/jdk9/dev/jdk
Compare against version: qparent
Summary of changes: 266 lines changed: 223 ins; 22 del; 21 mod; 5595 unchg
Patch of changes: <jdk.patch>
Bug id: JDK-8048209 : SynchronizedNavigableSet tailSet uses wrong mutex
Legend: Modified file Deleted file New file

[Cdiffs](src/share/classes/java/util/Collections.java.cdiff.html) [Udiffs](src/share/classes/java/util/Collections.java.udiff.html) [Sdiffs](src/share/classes/java/util/Collections.java.sdiff.html) [Frames](src/share/classes/java/util/Collections.java.frames.html) [Old](src/share/classes/java/util/Collections.java-.html) [New](src/share/classes/java/util/Collections.java.html) [Patch](src/share/classes/java/util/Collections.java.patch) [Raw](raw%5Ffiles/new/src/share/classes/java/util/Collections.java) src/share/classes/java/util/Collections.java

rev 10370 : 8048209: Collections.synchronizedNavigableSet().tailSet(Object,boolean) synchronizes on wrong object Reviewed-by: duke

1 line changed: 0 ins; 0 del; 1 mod; 5562 unchg

[Cdiffs](test/java/util/Collections/SyncSubMutexes.java.cdiff.html) [Udiffs](test/java/util/Collections/SyncSubMutexes.java.udiff.html) [Sdiffs](test/java/util/Collections/SyncSubMutexes.java.sdiff.html) [Frames](test/java/util/Collections/SyncSubMutexes.java.frames.html) [Old](test/java/util/Collections/SyncSubMutexes.java-.html) [New](test/java/util/Collections/SyncSubMutexes.java.html) [Patch](test/java/util/Collections/SyncSubMutexes.java.patch) [Raw](raw%5Ffiles/new/test/java/util/Collections/SyncSubMutexes.java) test/java/util/Collections/SyncSubMutexes.java (was test/java/util/Collections/ViewSynch.java)

rev 10370 : 8048209: Collections.synchronizedNavigableSet().tailSet(Object,boolean) synchronizes on wrong object Reviewed-by: duke

265 lines changed: 223 ins; 22 del; 20 mod; 33 unchg


This code review page was prepared using /home/mike/code/jdk/webrev/webrev.ksh(vers 25.6-hg+openjdk.java.net).