Jean-Marc Lasgouttes wrote:
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
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 double checked and it does check for idx. However, I see now that
any container implements lexicographic ordering tests. Question: would
the following patch make sense? Abdel, I know I have removed code
you just added, but I am wondering why we have to declare those extra
things. Also, it seems to me that the following could be removed:
+inline
+bool operator<(DocIterator const & di1, DocIterator const & di2)
+{
+ return di1.slices_ < di2.slices_;
+}
Wait, if the slices_ vectors have different size that won't work.
Abdel.
