[PATCH] 8202521: Add overloaded methods of Map#compute, Map#computeIfAbsent, Map#computeIfPresent

2018-05-03 Thread Jacob Glickman
Hello! This is my first patch, so I apologize in advance if I've done anything incorrectly (including botching the formatting of the mercurial diff below). I think it would be beneficial to have an overloaded `Map.computeIfAbsent` method where the computed value does not depend on the key; for th

[PATCH] 8202521: Add overloaded methods of Map#compute, Map#computeIfAbsent, Map#computeIfPresent

2018-05-08 Thread Jacob Glickman
Hi Paul, Thank you for the response! I understand I should have discussed this before submitting a patch, and will do so from now on. I would be happy to contribute to fixing some starter bugs, and I'd be very grateful if you are able point me to some! - Jacob

Re: [PATCH] 8202521: Add overloaded methods of Map#compute, Map#computeIfAbsent, Map#computeIfPresent

2018-05-14 Thread Paul Sandoz
Hi Jacob, I do have one idea (that i don’t think is currently represented as a bug, though i have not searched JBS), if you are willing to take it on. It will require some investigation, and careful testing, it’s not necessarily a starter bug :-), but i can help guide. Investigate new methods

Re: [PATCH] 8202521: Add overloaded methods of Map#compute, Map#computeIfAbsent, Map#computeIfPresent

2018-05-08 Thread Paul Sandoz
Hi Jacob, Thanks for looking at this. What you have done looks complete (although it is not necessary on HashTable since it tunnels directly to computeIfAbsent), and the performance evaluation is appreciated. While i have some sympathy with this, having written many such computeIfAbsent calls,