jdk (original) (raw)

Code Review for jdk

Prepared by: mduigou on Thu Apr 11 12:56:11 PDT 2013
Workspace: /home/mike/code/jdk/tl8/jdk
Compare against: ssh://hg.openjdk.java.net/jdk8/tl-gate/jdk
Compare against version: qparent
Summary of changes: 1977 lines changed: 1836 ins; 51 del; 90 mod; 7130 unchg
Changeset: <jdk.changeset>
Bug id: JDK-8010122 : Atomic update methods on Map and ConcurrentMap
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) -----[Raw](raw%5Ffiles/new/src/share/classes/java/util/Collections.java) src/share/classes/java/util/Collections.java

rev 6858 : 8004518: Add in-place operations to Map 8010122: Add defaults for ConcurrentMap operations to Map Reviewed-by: darcy, briangoetz, mduigou, dholmes, ulfzibis Contributed-by: Doug Lea

, Henry Jen henry.jen@oracle.com, Akhil Arora akhil.arora@oracle.com, Peter Levart peter.levart@gmail.com

296 lines changed: 293 ins; 0 del; 3 mod; 4152 unchg

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

rev 6858 : 8004518: Add in-place operations to Map 8010122: Add defaults for ConcurrentMap operations to Map Reviewed-by: darcy, briangoetz, mduigou, dholmes, ulfzibis Contributed-by: Doug Lea

, Henry Jen henry.jen@oracle.com, Akhil Arora akhil.arora@oracle.com, Peter Levart peter.levart@gmail.com

244 lines changed: 243 ins; 0 del; 1 mod; 1098 unchg

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

rev 6858 : 8004518: Add in-place operations to Map 8010122: Add defaults for ConcurrentMap operations to Map Reviewed-by: darcy, briangoetz, mduigou, dholmes, ulfzibis Contributed-by: Doug Lea

, Henry Jen henry.jen@oracle.com, Akhil Arora akhil.arora@oracle.com, Peter Levart peter.levart@gmail.com

260 lines changed: 244 ins; 14 del; 2 mod; 1165 unchg

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

rev 6858 : 8004518: Add in-place operations to Map 8010122: Add defaults for ConcurrentMap operations to Map Reviewed-by: darcy, briangoetz, mduigou, dholmes, ulfzibis Contributed-by: Doug Lea

, Henry Jen henry.jen@oracle.com, Akhil Arora akhil.arora@oracle.com, Peter Levart peter.levart@gmail.com

598 lines changed: 597 ins; 0 del; 1 mod; 477 unchg

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

rev 6858 : 8004518: Add in-place operations to Map 8010122: Add defaults for ConcurrentMap operations to Map Reviewed-by: darcy, briangoetz, mduigou, dholmes, ulfzibis Contributed-by: Doug Lea

, Henry Jen henry.jen@oracle.com, Akhil Arora akhil.arora@oracle.com, Peter Levart peter.levart@gmail.com

26 lines changed: 15 ins; 0 del; 11 mod; 160 unchg

[Cdiffs](test/java/util/Map/Defaults.java.cdiff.html) [Udiffs](test/java/util/Map/Defaults.java.udiff.html) [Sdiffs](test/java/util/Map/Defaults.java.sdiff.html) [Frames](test/java/util/Map/Defaults.java.frames.html) [Old](test/java/util/Map/Defaults.java-.html) [New](test/java/util/Map/Defaults.java.html) -----[Raw](raw%5Ffiles/new/test/java/util/Map/Defaults.java) test/java/util/Map/Defaults.java (was test/java/util/Map/ToArray.java)

rev 6858 : 8004518: Add in-place operations to Map 8010122: Add defaults for ConcurrentMap operations to Map Reviewed-by: darcy, briangoetz, mduigou, dholmes, ulfzibis Contributed-by: Doug Lea

, Henry Jen henry.jen@oracle.com, Akhil Arora akhil.arora@oracle.com, Peter Levart peter.levart@gmail.com

553 lines changed: 444 ins; 37 del; 72 mod; 78 unchg


This code review page was prepared using ../make/scripts/webrev.ksh(vers 23.18-hg+jbs).