Review Request 115713: Remove #if HAVE_X11 blocks from kstatusnotifieritem

Martin Gräßlin mgraesslin at kde.org
Thu Feb 13 06:53:41 UTC 2014


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

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/20140213/45f3da11/attachment.html>


More information about the Kde-frameworks-devel mailing list