Code Review for compute-once

Prepared by:Martin Buchholz (martin) on Fri Nov 1 09:02:48 PDT 2019
Summary of changes: 123 lines changed: 93 ins; 14 del; 16 mod; 12331 unchg
Changeset: compute-once.patch
Legend: Modified file Deleted file New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/util/concurrent/ConcurrentHashMap.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
21 lines changed: 8 ins; 0 del; 13 mod; 6361 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/ConcurrentHashMapTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
2 lines changed: 0 ins; 2 del; 0 mod; 882 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/ConcurrentSkipListMapTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
3 lines changed: 1 ins; 2 del; 0 mod; 1335 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/HashMapTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
2 lines changed: 0 ins; 2 del; 0 mod; 58 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/HashtableTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
2 lines changed: 0 ins; 2 del; 0 mod; 54 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/JSR166TestCase.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
8 lines changed: 7 ins; 0 del; 1 mod; 2185 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/LinkedHashMapTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
2 lines changed: 0 ins; 2 del; 0 mod; 58 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/MapImplementation.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
8 lines changed: 6 ins; 0 del; 2 mod; 47 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/MapTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
73 lines changed: 71 ins; 2 del; 0 mod; 231 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/concurrent/tck/TreeMapTest.java 

8231592: Clarify that ConcurrentHashMap compute methods mapping functions execute at most once
Reviewed-by: martin
2 lines changed: 0 ins; 2 del; 0 mod; 1120 unchg