Andre Poenitz wrote:

> On Sat, Mar 27, 2004 at 12:56:03AM +0100, Alfredo Braunstein wrote:
>> > Missing insetiterator.C?
>> 
>> No, just DocIterator::forwardInset in dociterator.C.
> 
>   
> void DocumentIterator::forwardInset()
> {
> forwardPos();
> while (!empty() && (pos() == lastpos() || nextInset() == 0))
> forwardPos();
> }
> 
> This is inefficient for texted as it iterates over all possible cursor
> positions and checks whether there is an inset there. It should be more
> efficient to use the insetlist somehow.

I think it is a complete no-issue for now, as iterating over all cursor
positions of the Userguide20 is still unnoticeable (sub second) here. If it
shows up in the profiles we can do it later. I'd really prefer to have the
simplest implementation now...

Alfredo


Reply via email to