Review Request 114922: Add runtime windowing system platform support to KWindowEffects

Martin Gräßlin mgraesslin at kde.org
Fri Jan 17 07:19:11 UTC 2014


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

(Updated Jan. 17, 2014, 7:19 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Repository: kwindowsystem


Description
-------

So far KWindowEffects had a compile time switch. Either it got compiled
for X11 or a dummy implementation got used. The actual runtime platform
got not considered at all. This change ensures that the X11
implementation is only used if the runtime platform is xcb. So if for
example one uses Wayland, but KWindowSystem got compiled with X11
support the dummy implementation is picked.
    
To achieve that a new pure virtual KWindowEffectsPrivate class is
introduced which has two implementations: KWindowEffectsPrivateDummy
and KWindowEffectsPrivateX11.
    
A new implementation for KWindowEffects is added which uses a global
static which holds a pointer to the KWindowEffectsPrivate. Thus the
global static functions acts as a factory method to pick the proper
implementation for the runtime system. The implementation of the
methods just delegate to the respective method of
KWindowEffectsPrivate.


Diffs
-----

  src/CMakeLists.txt 6a500fe9b257d8ff1871e799f7cdcd880de06128 
  src/kwindoweffects.cpp PRE-CREATION 
  src/kwindoweffects_dummy.cpp ce3b9bf4405d1befe3e36d3ede0a4bafe81d850d 
  src/kwindoweffects_p.h PRE-CREATION 
  src/kwindoweffects_x11.cpp b71e6b9f30d2f02ea86aa58b0b135a206904af7b 

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


Testing
-------

X11 unit test is working


Thanks,

Martin Gräßlin

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


More information about the Kde-frameworks-devel mailing list