Review Request 124226: Delete old highlighter

Lasse Liehu lasse.liehu at gmail.com
Sat Jul 11 08:18:47 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124226/
-----------------------------------------------------------

(Updated July 11, 2015, 8:18 a.m.)


Review request for KDE Frameworks and Laurent Montel.


Changes
-------

Removed unnecessary if.


Repository: ktextwidgets


Description
-------

According to SpellCheckDecorator::setHighlighter's documentation, the old automatically created default highlighter must be manually deleted.

This fixes bug 343459 in Lokalize.


Diffs (updated)
-----

  src/widgets/ktextedit.cpp e4bd41d 

Diff: https://git.reviewboard.kde.org/r/124226/diff/


Testing
-------

Tested that it works in Lokalize. Also ran tests.


Thanks,

Lasse Liehu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150711/ff46d693/attachment.html>


More information about the Kde-frameworks-devel mailing list