On 12/16/2010 03:43 AM, Pavel Sanda wrote:
hi,

the attached patch has profound impact on the speed of moving with cursor
and typing as well. at least here.

if you wonder how to reproduce the slowness problems on your box, try to open
user guide, set the nesting on deepest level and try to type fastly or move
with cursor, maybe you'll be lucky...

opinions?

Provided that it doesn't introduce regression, it looks good. But I would rather put a shorter timing here in order to avoid inconcistencies and crashes (ex: if the user is fast enough to use the outine one second after typing a char). I am sure you won't feel the difference if you set it to 0.5s. That's because the main improvment of your patch is that it prunes out uneeded updateView() calls.

Abdel.

Reply via email to