Code Review for jdk

Prepared by:mduigou on Wed Apr 10 22:36:59 PDT 2013
Workspace:/home/mike/code/jdk/tl8/jdk
Compare against: ssh://hg.openjdk.java.net/jdk8/tl-gate/jdk
Compare against version:6857
Summary of changes: 1773 lines changed: 1643 ins; 51 del; 79 mod; 7141 unchg
Changeset: jdk.changeset
Bug id: JDK-8010122 : Atomic update methods on Map and ConcurrentMap
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw 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 <dl at cs.oswego.edu>, Henry Jen <henry.jen@oracle.com>, Akhil Arora <akhil.arora@oracle.com>, Peter Levart <peter.levart@gmail.com>
94 lines changed: 91 ins; 0 del; 3 mod; 4152 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw 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 <dl at cs.oswego.edu>, 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 Udiffs Sdiffs Frames Old New ----- Raw 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 <dl at cs.oswego.edu>, 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 Udiffs Sdiffs Frames Old New ----- Raw 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 <dl at cs.oswego.edu>, Henry Jen <henry.jen@oracle.com>, Akhil Arora <akhil.arora@oracle.com>, Peter Levart <peter.levart@gmail.com>
607 lines changed: 606 ins; 0 del; 1 mod; 477 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw 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 <dl at cs.oswego.edu>, Henry Jen <henry.jen@oracle.com>, Akhil Arora <akhil.arora@oracle.com>, Peter Levart <peter.levart@gmail.com>
15 lines changed: 15 ins; 0 del; 0 mod; 171 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw 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 <dl at cs.oswego.edu>, 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).