Andre Poenitz <[EMAIL PROTECTED]> writes:

| This is a patch to use 'par_type' offsets to ownerParagraphs().begin()
| in LyXCursor.
>
| The drawback is obvious: we have O(n) access in some cases where we had
| O(1) before.

Only some cases?

| However,I have not seen _any_ indication that this is
| noticable after playing around with the UserGuide.
>
| In any case, the benefits seem to clearly outweigh this: LyXCursor is
| now 'copiable' and we can add asserts to the 'getPar()' set of functions
| to catch accesses to 'uninitialized' cursor data early. And we've lost a
| few dependency on ParagraphList_fwd.h.

I have no idea what 'copiable' is. The cursor is perfectly copiable
with the iterator so what does this really change?


-- 
        Lgb

Reply via email to