Code Review for jdk

Prepared by:mduigou on Mon Jun 17 16:25:10 PDT 2013
Workspace:/home/mike/code/jdk/tl8/jdk
Compare against: ssh://hg.openjdk.java.net/jdk8/tl-gate/jdk
Compare against version:7359
Summary of changes: 491 lines changed: 409 ins; 12 del; 70 mod; 12520 unchg
Changeset: jdk.changeset
Bug id: JDK-8016446 : HashMap/IdentityHashMap should implement forEach(BiConsumer<? super K, ? super V> action)
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/HashMap.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
112 lines changed: 106 ins; 3 del; 3 mod; 3006 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/Hashtable.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
28 lines changed: 20 ins; 0 del; 8 mod; 1439 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/IdentityHashMap.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
34 lines changed: 34 ins; 0 del; 0 mod; 1558 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/LinkedHashMap.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
28 lines changed: 28 ins; 0 del; 0 mod; 473 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/Map.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
13 lines changed: 12 ins; 1 del; 0 mod; 1090 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/TreeMap.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
29 lines changed: 29 ins; 0 del; 0 mod; 2921 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/WeakHashMap.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
44 lines changed: 44 ins; 0 del; 0 mod; 1320 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/java/util/concurrent/ConcurrentMap.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
34 lines changed: 34 ins; 0 del; 0 mod; 186 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/java/util/Map/Defaults.java

rev 7360 : 8016446: Improve forEach/replaceAll for Map, HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
Reviewed-by: forax, duigou, psandoz
Contributed-by: Mike Duigou <mike.duigou@oracle.com>, Remi Forax <forax@univ-mlv.fr>
169 lines changed: 102 ins; 8 del; 59 mod; 527 unchg

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