You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@groovy.apache.org by GitBox <gi...@apache.org> on 2019/04/02 18:21:58 UTC

[GitHub] [groovy] eric-milles commented on issue #903: GROOVY-9060: Make the initial capacity of LinkedHashMap be power of 2…

eric-milles commented on issue #903: GROOVY-9060: Make the initial capacity of LinkedHashMap be power of 2…
URL: https://github.com/apache/groovy/pull/903#issuecomment-479132066
 
 
   Doesn't this just move the math from inside of HashMap to InvokerHelper.initialCapacity?  Where is the performance gain?  Is there any benchmark for this?
   
   Did the constant {{LN2}} get used anywhere?  It appears to be added but unused.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services