On 10/08/2009 23:03, Vincent van Ravesteijn wrote:
Abdelrazak Younes schreef:
On 10/08/2009 22:57, Vincent van Ravesteijn wrote:
you...@lyx.org schreef:
Author: younes
Date: Mon Aug 10 22:54:22 2009
New Revision: 30970
URL: http://www.lyx.org/trac/changeset/30970

Log:
Move the inline spellchecking code from collectWords() to the row drawing routine. The text row is spell checked only if changed!

Now inline SpellChecker is faster than ever. Now, I honestly think that we blow out all the competitors (thunderbird, OO, MSWord, etc) :-)

SpellChecking in the RowPainter code ?

Against your religion?

Abdel.

Let's do updateLabels in the rowPainter, and updateMacros, and setBuffer ?

Let's be a little bit pragmatic here:
1) Even if this is not obvious, this spellckecker code has something to do with drawing: the purpose is precisely to draw spellchecking marks. 2) There is absolutely no need to draw spellchecking marks outside of the visible text area
3) There is no need to redo spellchecking if the row contents did not change
4) the row painting code is the *only* place where we check if the contents has changed.

Abdel.

Reply via email to