Thilo Goetz wrote:
Jörn Kottmann wrote:
This methods calls Collections.binarySearch.
I doubt that the call is correct since binarySearch
assumes that the passed objects are Comparable which
FeatureStructure is not.
I have seen similar code in Subiterator.moveTo where I think its correct.
Are UnambiguousIteratorImpl and Subiterator only made for AnnotationFS
objects ?
Yes. Does that make it correct?
No, Subiterator calls a different binarySearch method which takes
a Comparator. We should then do the same in UnambiguousIteratorImpl.
Is there a test for this iterator ? For me it looks like that the moveTo
method
is broken.
Jörn
