On Tue, 19 Jul 2022 15:48:09 GMT, Andy Goryachev <[email protected]> wrote:
>> - added missing hashCode() methods > > Andy Goryachev has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains five additional > commits since the last revision: > > - 8289389: review comments > - Merge remote-tracking branch 'origin/master' into 8289389.hash > - 8289389: minimize the impact of collision > - 8289389: toExternalForm() > - 8289389: implemented missing hashCode() Yes, it's beyond the scope of PR. I suppose people will have to be careful as with all classes that are mutable and have decided to override equals/hashCode -- no reason to stop now I suppose. ------------- PR: https://git.openjdk.org/jfx/pull/821
