On Thu, 29 Apr 2021 07:05:18 GMT, Mitsuru kariya 
<github.com+2217224+kariya-mits...@openjdk.org> wrote:

>> The current `hashCode` implementation of SA's Address subclasses ignores the 
>> upper 32 bits of the long value.
>> This PR changes to use `Long.hashCode` instead.
>
> Mitsuru kariya has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update copyright

Thanks, yes it can definitely be an update to this change.

We can update the bug title, as I notice that is specific to Address classes, 
but as there are only a few more methods doing the same truncation, it makes 
sense to include them. e.g.  "Some SA classes could use better hashCode() 
implementation."

-------------

PR: https://git.openjdk.java.net/jdk/pull/3522

Reply via email to