I just noticed that the javadocs for org.apache.pivot.collections.Sequence state that public int remove(T item); will return 'The index of the item that was removed, or -1 if the item could not be found.' http://svn.apache.org/repos/asf/pivot/trunk/core/src/org/apache/pivot/collections/Sequence.java
However, both org.apache.pivot.collections.concurrent.SynchronizedList & org.apache.pivot.collections.adapter.ListAdapter throw IllegalArgumentExceptions rather than returning -1. (Those are the only two problematic implementations of Sequence which I could find.) I can't see any code that relies on this behaviour so it looks safe remove the Exception throwing. Regards, Chris
