Jörn Kottmann wrote: > 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
I'll check, but I may not get to it till tomorrow. --Thilo
