On Wed, May 16, 2007 at 10:37:52AM +0200, Jean-Marc Lasgouttes wrote:
> >>>>> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
>
> Abdelrazak> Andre Poenitz wrote:
> >> The canonical solution wold look like the following (completely
> >> untested!)
> >>
> >> bool operator<(DocIterator const & di1, DocIterator const & di2) {
> >> size_t const n1 = di1.slices_.size(); size_t const n2 =
> >> di2.slices_.size(); for (size_t i = 0; i < n1 && i < n2; ++i) {
> >> Slice const & sl1 = di1.slices_[i]; Slice const & sl2 =
> >> di2.slices_[i]; if (sl1.idx() != sl2.idx()) return sl1.idx() <
> >> sl2.idx();
>
> Abdelrazak> Hum the version I committed from JMarc used the
> Abdelrazak> CursorSlice::operator<() which does not check for idx()
> Abdelrazak> IIRC. Should add this test JMarc?
>
> I guess the test should be added to CursorSlice::operator<().
>
> Andre'? This is your code, right?
It belongs there, yes.
Andre'
