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

Alex Merry kde at randomguy3.me.uk
Thu Jan 9 19:24:13 UTC 2014


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

Ship it!


All looks sane to me (and I can't see anything obvious that would cause any regressions).


src/kwindoweffects_dummy.cpp
<https://git.reviewboard.kde.org/r/114922/#comment33579>

    Sort of orthogonal to the change, but is it worth doing windowSizes.reserve(ids.count()) here?


- Alex Merry


On Jan. 9, 2014, 9:03 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114922/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2014, 9:03 a.m.)
> 
> 
> 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/20140109/a2b50020/attachment.html>


More information about the Kde-frameworks-devel mailing list