Community
Participate
Working Groups
Build Identifier: I have an issue where I'm asking a DecoratingObservableCollection whether it equals() itself (when refreshing the input on a TableViewer), and it's reporting no. The implementation currently delegates the equals method to the decorated object, but of course does not attempt to unwrap the parameter to equals. The decorated list (a HibernatePersistableEList) does a simple object identity check, which fails because it's comparing the wrapped list with the wrapper. This simple patch would fix this case, and I can't see any harm. I'm still hazy on the auto-tracking of getters, so I'm not sure if I should avoid calling getterCalled() unless we actually delegate the call. public boolean equals(Object o) { getterCalled(); if (this == o) return true; return decorated.equals(o); } Reproducible: Always
I've added a test case to my databinding sample project at github (http://github.com/ajuckel/org.example.emfdb/commit/b533ae1fd6b9860423927878d45ae6519e9a1b84). As an alternative test, we could update StructuredViewer#equals(Object, Object) to be: protected boolean equals(Object elementA, Object elementB) { if (elementA == elementB) { return true; } else if (elementA == null) { return false; } else if (comparer == null) { return elementA.equals(elementB); } else { return comparer.equals(elementA, elementB); } } The elementA == elementB test captures both the previously tested both-are-null case, as well as my desire for identical objects to always report equality.
This is a safe change, I'll go ahead and push this. (In reply to comment #1) > As an alternative test, we could update StructuredViewer#equals(Object, Object) > to be: > > protected boolean equals(Object elementA, Object elementB) { > if (elementA == elementB) { > return true; > } else if (elementA == null) { > return false; > } else if (comparer == null) { > return elementA.equals(elementB); > } else { > return comparer.equals(elementA, elementB); > } > } One workaround would be to call StructuredViewer.setComparer(IElementComparer), and provide a comparer that does what you've outlined above.
Created attachment 176957 [details] Patch
Released to HEAD > 20100818
