Review Request 115713: Remove #if HAVE_X11 blocks from kstatusnotifieritem
Martin Gräßlin
mgraesslin at kde.org
Sat Feb 15 12:23:22 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115713/
-----------------------------------------------------------
(Updated Feb. 15, 2014, 12:23 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks, kdewin and Marco Martin.
Repository: knotifications
Description
-------
Remove #if HAVE_X11 blocks from kstatusnotifieritem
They are all only around kwindowsystem framework usage which is not
needed to have in ifdefs. The runtime usage might differ, but from
looking at each line of code it works correctly. It's mostly about
virtual desktops and thus calls are simply ignored on systems not
supporting virtual desktops.
Diffs
-----
src/kstatusnotifieritem.cpp 82c4e65e1869fe00432634bcabbf410d44e4a80a
Diff: https://git.reviewboard.kde.org/r/115713/diff/
Testing
-------
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140215/7b43c4f3/attachment.html>
More information about the Kde-frameworks-devel
mailing list