On Fri, 4 Aug 2023 19:54:58 GMT, Sean Mullan <mul...@openjdk.org> wrote:

> I would add to that list that I think it is really important any change to 
> the `hashCode` impl doesn't break the equals/hashCode contract, so it's 
> important to look at the `equals` implementation in conjunction with any 
> changes to `hashCode`, mostly to ensure 2 equal objects continue to have the 
> same hash code.

True. I skipped that criteria, as I thought it should go without saying.

> It is also important to question any oddity about the `hashCode` impl, like I 
> did for DESKey and try to understand why it may have been implemented that 
> way, such as skipping the first byte.

True again. If you have any other examples where the changes to hashCode look 
suspicious, please tell me so.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14738#discussion_r1284837771

Reply via email to