On 23-Apr-2002 jug wrote:

> Modified files:
>       lyx-devel/src/: BufferView.C BufferView.h BufferView_pimpl.C 
>                       BufferView_pimpl.h ChangeLog 
> 
> Log message:
>       Remove wrong and unneeded insetSleep/Wakeup functions.

This fixes the bug that the cursor is set to the wrong inset on a buffer
change. IMO this are old functions when we still had the_locking_inset
inside BV and so they are needed, now TLI is inside LyXText so we don't
need them anymore (and you'll see that all works as expected).

What IS wrong is that if we have more than 1 buffer loaded and change
the BV-Width all but the acutal visible LyXText are deleted and with this
we loose all information saved therein (CursorPos, TLI, Selections). This
is IMO wrong! We shouldn't delete the LyXTexts in the textlist, but we
should also not update them to the new width as otherwise it would take
maybe too long if we have to recalculate all loaded buffers. We just should
flag them and on buffer/change resize them. Doing so we wouldn't loose any
information.

I don't know if we want to do this now or later but it has to be done and
if we don't do it before 1.2.0 than at least we should open a bug for this.
(well I anyway open a bug so it is there and I will assign it to Lars
otherwise he will get bored with no work to do ;)

           Jug

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._
Dr. Jürgen Vigna        E-Mail:  [EMAIL PROTECTED]
Italienallee 13/N       Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen           Web:     http://www.sad.it/~jug
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

The majority of husbands remind me of an orangutang trying to play the violin.
                -- Honor'_e DeBalzac

Reply via email to