From 7e1185e1529091941f6baa0d801e656a7da0cf4d Mon Sep 17 00:00:00 2001 From: Stuart Marks Date: Mon, 16 Jun 2014 17:52:21 -0700 Subject: [PATCH] 8044730: small errors in ConcurrentHashMap and LongAdder docs Reviewed-by: martin --- .../share/classes/java/util/concurrent/ConcurrentHashMap.java | 4 ++-- .../share/classes/java/util/concurrent/atomic/LongAdder.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/jdk/src/share/classes/java/util/concurrent/ConcurrentHashMap.java b/jdk/src/share/classes/java/util/concurrent/ConcurrentHashMap.java index 880ecb2bbdd..5dd5d4a846a 100644 --- a/jdk/src/share/classes/java/util/concurrent/ConcurrentHashMap.java +++ b/jdk/src/share/classes/java/util/concurrent/ConcurrentHashMap.java @@ -133,12 +133,12 @@ import java.util.stream.Stream; * mapped values are (perhaps transiently) not used or all take the * same mapping value. * - *

A ConcurrentHashMap can be used as scalable frequency map (a + *

A ConcurrentHashMap can be used as a scalable frequency map (a * form of histogram or multiset) by using {@link * java.util.concurrent.atomic.LongAdder} values and initializing via * {@link #computeIfAbsent computeIfAbsent}. For example, to add a count * to a {@code ConcurrentHashMap freqs}, you can use - * {@code freqs.computeIfAbsent(k -> new LongAdder()).increment();} + * {@code freqs.computeIfAbsent(key, k -> new LongAdder()).increment();} * *

This class and its views and iterators implement all of the * optional methods of the {@link Map} and {@link Iterator} diff --git a/jdk/src/share/classes/java/util/concurrent/atomic/LongAdder.java b/jdk/src/share/classes/java/util/concurrent/atomic/LongAdder.java index e7415dea821..6810edf0010 100644 --- a/jdk/src/share/classes/java/util/concurrent/atomic/LongAdder.java +++ b/jdk/src/share/classes/java/util/concurrent/atomic/LongAdder.java @@ -57,7 +57,7 @@ import java.io.Serializable; * frequency map (a form of histogram or multiset). For example, to * add a count to a {@code ConcurrentHashMap freqs}, * initializing if not already present, you can use {@code - * freqs.computeIfAbsent(k -> new LongAdder()).increment();} + * freqs.computeIfAbsent(key, k -> new LongAdder()).increment();} * *

This class extends {@link Number}, but does not define * methods such as {@code equals}, {@code hashCode} and {@code