gnodet commented on a change in pull request #125:
URL: https://github.com/apache/maven-resolver/pull/125#discussion_r723887906
##########
File path:
maven-resolver-api/src/main/java/org/eclipse/aether/DefaultSessionData.java
##########
@@ -81,4 +82,8 @@ public Object get( Object key )
return data.get( key );
}
+ public Object computeIfAbsent( Object key, Function<Object, Object>
mappingFunction )
Review comment:
@rmannibucau @cstamas honestly, I agree with you. I've used the
`computeIfAbsent` method on the map a lot of time, and I've always been puzzled
about the inclusion of the key in the signature which is most of the time
useless. I wanted to keep the `Map` compatibility, but if you guys don't think
it's necessary, I'm more than happy to just use a `Supplier`...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
