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).