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