Review Request 115230: Add platform check to KSelectionOwner and KSelectionWatcher

[Martin Gräßlin]

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

(Updated Jan. 27, 2014, 9:54 a.m.)


Review request for KDE Frameworks.


Changes
-------

Added warning in the factory methods in case it's not X11.


Repository: kwindowsystem


Description
-------

Add platform check to KSelectionOwner and KSelectionWatcher

These are highly X11 specific classes and don't make any sense on a
platform other than xcb and were potentially crashy by using QX11Info
without verifying that the platform is used.

The implementation will now only create the d-ptr in case that the
code is running on the xcb platform and ensures that no call to xcb
is done unconditionally. All methods perform an early return in case
the d-ptr is null. Non-void methods return a sensible default value
in this case.


Diffs (updated)
-----

  src/kselectionowner.h 512bd5ebc6336c3a07d78dd964599d3f48d8eb33 
  src/kselectionwatcher.cpp 87e0a9612f2bc6ced3de7c31797411b1eb183d41 
  src/kselectionwatcher.h 798fe70bfc8175764f3fc3d09de97d971d5e57ad 
  src/kselectionowner.cpp 3715267e280822ba541196d9c63fd28f58cdc7ee 

Diff: https://git.reviewboard.kde.org/r/115230/diff/


Testing
-------

unit test is not failing. No test on non-X11, though.


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140127/b06fc6ba/attachment.html>