[Konsole-devel] Review Request 128346: Port to QRegularExpression
Martin Tobias Holmedahl Sandsmark
martin.sandsmark at kde.org
Sat Jul 2 19:42:07 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128346/
-----------------------------------------------------------
(Updated July 2, 2016, 7:42 p.m.)
Review request for Konsole and Kurt Hindenburg.
Changes
-------
Removed a spurious QDebug include.
Repository: konsole
Description
-------
Port away from using QRegExp to QRegularExpression. The result is a bit better performance, a bit more readable code (imho) and also no more relying on mutable state in static objects (like in the keyboard translator parser).
Diffs (updated)
-----
src/Filter.h 8d125f2
src/Filter.cpp 789c4cd
src/KeyboardTranslator.cpp 3effb72
src/Profile.cpp 287b047
src/SessionController.h 32a9e7c
src/SessionController.cpp b37937d
Diff: https://git.reviewboard.kde.org/r/128346/diff/
Testing
-------
Tested all the affected code individually.
Thanks,
Martin Tobias Holmedahl Sandsmark
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20160702/30683cf2/attachment.html>
More information about the konsole-devel
mailing list