On Fri, 2005-05-06 at 15:48, Angus Leeming wrote:

...

> > This fixes the first crash reported by Juergen.
> > 
> > It turns out BufferView::update() can in fact be called multiple times
> > at the same time, if you press keys fast enough. I wonder if this is
> > supposed to be. Anyway, making the handling of the coordinate cache
> > re-entrant fixes the bug. This requires "updating" to be a counter
> > rather than a bool.
> > 
> > Tell me if I should be happy with this; I'm not so sure.
> 
> (Looking at the correct attachment...)
> 
> This patch looks completely reasonable. It would be interesting to find 
> out the source of the bv.update() within bv.update() calls. One candidate 
> would be the graphics loading stuff which report their changed status by 
> calling
>      LyX::cref().updateInset(this);
> which ends up calling bv.update().

But there is no graphics or the like within a mile.
It is _typing characters_ that does this. Does that ring a bell?

- Martin

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to