[ 
https://issues.apache.org/jira/browse/MATH-786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13269493#comment-13269493
 ] 

Thomas Neidhart commented on MATH-786:
--------------------------------------

Do you have a use case at hand that requires this change?

In general, I would oppose something like that as it sounds like premature 
optimization.
Although documenting the behavior in javadoc is correct, it may still create 
problems and makes debugging more difficult.

OTOH, if the default is "false" and only set explicitly to "true" in specific 
cases where a user / developer knows exactly what he/she is doing it would be 
fine I guess.
                
> "hashCode" in "Pair" class
> --------------------------
>
>                 Key: MATH-786
>                 URL: https://issues.apache.org/jira/browse/MATH-786
>             Project: Commons Math
>          Issue Type: Improvement
>    Affects Versions: 3.0
>            Reporter: Gilles
>            Assignee: Gilles
>            Priority: Trivial
>             Fix For: 3.1
>
>
> Since "Pair" is supposed to be an immutable class, couldn't we cache the 
> "hashCode" value at construction? That would supposedly make it more 
> efficient when used in maps.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to