Description Maxim Gordienko 2011-07-13 21:27:07 EDT

Build Identifier: M20110210-1200

The indices in a ListDiff event fired by EObjectObservableList during a multi-remove operation (removeAll()/clear()) are incorrect.

The problem (as I can see it) is in the implementation of the
Notification.REMOVE_MANY case in the Adapter created in
EObjectObservableList#firstListenerAdded()

The index computation is incorrect. It should take the removed element
into account not just blindly increment the index. (see how the diff
is calculated in WritableList#clear())

Another problem is with clear(). The notification event index will be
set to Notification.NO_INDEX which is -1. (see
NotifyingListImpl#clear())

Reproducible: Always

Steps to Reproduce:
               Container container = FACTORY.createContainer();
               container.getComponents().add(FACTORY.createComponent());
               container.getComponents().add(FACTORY.createComponent());
               IObservableList list = EMFObservables.observeList(container,
PACKAGE.getContainer_Components());

               Listener listener = new Listener();
               list.addListChangeListener(listener);

               list.clear();

               assertEquals(Arrays.asList(0, 0), listener.getIndices());
               // The actual indices are [-1, 0]