Jean-Marc Lasgouttes wrote:
"Alfredo Braunstein" <[EMAIL PROTECTED]> writes:

Both I guess. That being said, I am not religious about that, you can
get rid of it if you find a proper replacement AFAIAC; but I don't think
there is a need to optimize anything here. There is a lot more
inefficient stuff in the source code.
This is possibly true. But changes here are easy and potentially
affect positively large portions of the code, other optimizations need
thinking ;-) Besides, blame JMarc, he started it ;-)

I was not thinking about it in terms of optimization, actually. Having
a real vector interface feels more natural than this mix between list
an vector that we have today.

Well, the interface is already pretty much a vector interface. I think you won't have to change a single line of code if you implement the vector<Paragraph *> based solution... If you talk about being able to use stuff like 'begin() + pos' then I strongly suggest to stay with std::advance (and boost::next/previous) instead. This convention is what enable us to change to whatever container we like without problems. Please keep it that way.

Abdel.

Reply via email to