Re: Java 8 RFR 8024331: j.u.Map.computeIfPresent() default/nondefault implementations don't throw NPE if the remappingFunction is null and the key is absent

[Mike Duigou]

The Map/Defaults.java changes only test HashMap. It should probably be extended 
to test most/all of the other implementations.

I have some worry that the superfluous assignment of the 
Objects.requireNonNull() result might have a non-negligible cost. 

Mike

On Sep 17 2013, at 13:49 , Brian Burkhalter wrote:

> Please review this proposed patch at your convenience.
> 
> Summary:      Explicitly check for null remappingFunction parameter (and 
> clean up similar code for consistency).
> Issue:                
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8024331
> Webrev:               http://cr.openjdk.java.net/~bpb/8024331/
> 
> Thanks,
> 
> Brian