Noted. Thank you. Mike
On Dec 17 2013, at 05:51 , Paul Sandoz <[email protected]> wrote: > > On Dec 17, 2013, at 4:42 AM, Mike Duigou <[email protected]> wrote: > >> I've updated the documentation per Paul's suggestions. Specifically, in >> addition to the existing put() and get() methods the new Map methods >> >> putIfAbsent() >> getOrDefault() >> compute() >> computeIfAbsent() >> computeIfPresent() >> merge() >> >> are all documented to perform a single access of the entry (assuming that an >> entry is created or modified as a result). >> >> replace() >> >> is documented as accessing the entry, if it exists and a replacement is >> made. If no replacement is made then no access is recorded. >> >> http://cr.openjdk.java.net/~mduigou/JDK-8029795/2/webrev/ >> > > Looking good. Just one v. minor thing for: > > + * invocation completes). The {@code replace} method only results in an > access > + * of the entry if the value is replaced. The {@code putAll} method > generates one > > There are two replace methods: > > * invocation completes). The {@code replace} methods only results in an access > > Paul.
