Alfredo Braunstein wrote:

> Jean-Marc Lasgouttes wrote:
> 
>> I took a quick look at CoordBranch and it seems to work well. I found
>> a few bugs that happen to be in HEAD too, but seems to be related to
>> what you are doing.
>> 
>> - load Tutorial; Ctrl-End ==> crash
>> 
>>   Assertion triggered in Point::Point(int, int) by failing check "y <
>>   4000" in file ../../lyx-devel/src/coordcache.h:29 Aborted
>> 
> 
> Humm... I don't see that here (in CoordBranch). Do you get every time, or
> only once in a while? Can you send a backtrace?

This is important. I don't understand why could we be drawing at such large
coordinates. Are you sure this is coordbranch? [btw, you don't happend to
have a > 4000 pixels high monitor, do you?]

>> - Selecting with the mouse is (still) very very slow; I suspect though
>>   that your work has nothing to do with that.
> 
> Well, this is something we should think about certainly. We do an update
> call on every MOTION event. I have some ideas to speed this up: for
> instance we could update only when the cursor has actually changed

I've commited this as was pretty easy. It doesn't seem to change things so
much unfortunately.

> position, or if that is not enough, make an update with a timer like on
> scrolling. In any case, it's not slower than HEAD, or is it?

I'm sure there are many other possible optimizations we could do. OTOH, I
don't think this is the right time for optimization.

Alfredo


Reply via email to