[kde-edu]: Review Request: Syntax Hightlighting in Cantor

Alexander Rieder alexanderrieder at gmail.com
Sun Aug 22 22:15:16 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/5079/#review7157
-----------------------------------------------------------


Hi,
i haven't looked at the code yet, but in its current form the patch doesn't compile, as you try to include the non-existent "dbhighlighter.h" in octavehighlighter.h and sagehighlighter.h, and "highlightdatabase.h" in octavebackend.cpp.

- Alexander


On 2010-08-22 14:03:57, Miha Cancula wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5079/
> -----------------------------------------------------------
> 
> (Updated 2010-08-22 14:03:57)
> 
> 
> Review request for KDE-Edu and Alexander Rieder.
> 
> 
> Summary
> -------
> 
> As discussed on email, I implemented a different approach to highlighting in Cantor. I'd like some feedback before committing it. 
> 
> I introduced additional API in DefaultHighlighter and moved most of the logic in it, so the individual backend-specific highlighters only specify conditions (either QString or QRegExp) and matching text formats. The code looks much cleaner this way.
> 
> As Alexander and Oleksiy already determined, breaking the text into words and looking for these words is faster than iterating over a huge list of regexes and looking for each of them in the text. That's why functions, variables and keywords are implemented this way. OTOH, thing like comments and strings are easier done using Regexes, so this functionality is still there.  
> 
> The implementation uses a QHash<QString, QTextCharFormat> and a QHash<QRegExp, QTextCharFormat>. If anyone knows of a way to make it faster, please say so. 
> 
> I also updated highlighters for Octave, Maxima and Sage to use the word-based API as much as possible. Most of their code was also removed, because it's now in DefaultHighlighter. I left R alone because Oleksiy's work is not yet in trunk. 
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/cantor/src/backends/maxima/maximahighlighter.h 1165930 
>   /trunk/KDE/kdeedu/cantor/src/backends/maxima/maximahighlighter.cpp 1165930 
>   /trunk/KDE/kdeedu/cantor/src/backends/octave/octavehighlighter.h 1165930 
>   /trunk/KDE/kdeedu/cantor/src/backends/octave/octavehighlighter.cpp 1165930 
>   /trunk/KDE/kdeedu/cantor/src/backends/sage/sagehighlighter.h 1165930 
>   /trunk/KDE/kdeedu/cantor/src/backends/sage/sagehighlighter.cpp 1165930 
>   /trunk/KDE/kdeedu/cantor/src/lib/defaulthighlighter.h 1165930 
>   /trunk/KDE/kdeedu/cantor/src/lib/defaulthighlighter.cpp 1165930 
> 
> Diff: http://reviewboard.kde.org/r/5079/diff
> 
> 
> Testing
> -------
> 
> I tested Maxima and Sage and they seem to be faster than before for large blocks. I used to have problems with non-smooth scrolling in Octave due to the large number of functions, but now it feels normal. I didn't notice any regressions (yet). 
> 
> It all works both on trunk and 4.4.  
> 
> 
> Thanks,
> 
> Miha
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-edu/attachments/20100822/013f7b42/attachment.htm 


More information about the kde-edu mailing list