>>>>> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:

Abdelrazak> Oh yes I do... The fix is really really stupid. I had the
Abdelrazak> order inversed in operator-:

This is pretty bad indeed :)

Abdelrazak> Please find attached the new RandomAccessList. I have also
Abdelrazak> erased all the operator=, they are not needed afterward.

Very good.

Abdelrazak> So now, I officially declare it ready for testing :-).
Abdelrazak> Jean-Marc, I know that this a big change in the algorithm
Abdelrazak> sense but it is very minimal in terms of code changes and
Abdelrazak> can be reverted very easily. What do you think in putting
Abdelrazak> it just after 1.4.1 so that people test it and revert to
Abdelrazak> vector just before 1.4.2 if any problem is detected?

What I had in mind was to apply it to trunk as soon as Lars accepts it
and then, if nothing bad appears, apply it for 1.4.2.

Neverthless, I am still not convinced that the inner list container is
useful. Since you have a vector<Paragraph*> already holding the data,
why bother having a second data structure?

JMarc

Reply via email to