Review Request 112519: trim KStyle
Wojciech Kapuscinski
wojtask9.kde at gmail.com
Fri Sep 6 15:11:44 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112519/
-----------------------------------------------------------
(Updated Sept. 6, 2013, 3:11 p.m.)
Review request for KDE Frameworks.
Changes
-------
Kill drawKStylePrimitive(), remove everything from eventFilter (see Christoph Feck comment), restore "runtime element extension".
Description
-------
KStyle 2/3: Remove everything from KStyle which is not a reimplementation of the following methods: polish, unpolish, eventFilter, styleHint, pixelMetrics, standardIcons.
Note:
Probably we can remove drawKStylePrimitive (and a lot enums/structs in kstyle.h) but right now I'm unsure if I understand KStyle::eventFilter correctly and this is material for 3/3 KStyle task :)
Diffs (updated)
-----
staging/frameworkintegration/src/kstyle/kstyle.h a3061d3
staging/frameworkintegration/src/kstyle/kstyle.cpp 17a1d41
Diff: http://git.reviewboard.kde.org/r/112519/diff/
Testing
-------
It builds and tests looks OK
Thanks,
Wojciech Kapuscinski
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130906/8534c5f5/attachment.html>
More information about the Kde-frameworks-devel
mailing list