On 11/05/2013 02:29 PM, Gary Gregory wrote:
> I'd like to see more docs on why AbstractPatriciaTrie implements hashCode()
> but not equals(). I see that AbstractPatriciaTrie inherits equals from
> AbstractMap, but why not also inherit hashCode()? What is confusing is that
> AbstractPatriciaTrie's hashCode uses a MyEntryIterator but why not inherit
> the behavior when AbstractMap calls entrySet().iterator() in its own
> hashCode method?
> 
> If our behavior is correct, we need better docs.

I do not know why this custom hashCode method has been introduced, but
it is the same as in AbstractMap (which it inherits).

As this overridden method does not bring anything, I would propose to
remove it, but for me this is certainly no blocker.

Thomas

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to