Review Request 116610: Use flag types in NETRootInfo
Martin Gräßlin
mgraesslin at kde.org
Mon Mar 17 08:03:20 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116610/
-----------------------------------------------------------
(Updated March 17, 2014, 8:03 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks.
Repository: kwindowsystem
Description
-------
Use flag types in NETRootInfo
This replaces the usage of the unsigned long[] array with
NET::Properties, NET::Properties2, NET::WindowTypes, NET::States and
NET::Actions mixed in - for the fun even in different order
depending on the ctor to use.
This includes the following changes:
* NETRootInfo::event(xcb_generic_event_t*) returns NET::Properties
* new overload added to NETRootInfo::event taking NET::Properties*
and NET::Properties2* as arguments
* existing overload for NETRootInfo::event taking usinged long* as
argument is deprecated and forwards to the new added overload
* NETRootInfo::passedProperties returns NET::Properties and new
methods for Properties2, WindowTypes, States and Actions are added.
This is a SC-break, but there is according to lxr no usage except
in the unit tests
* NETRootInfo::supportedProperties returns NET::Properties and new
methods for Properties2, WindowTypes, States and Actions are added.
This is a SC-break, but there is according to lxr no usage except
in the unit tests
* ctors taking unsigned long* is changed to taking the flags as
arguments. This in an SC-break, but the ctor broke already caused
by the change from Display* to xcb_connection_t*. The ctor for
window manager should only be used by KWin. Other ctor is also
mainly only used in kde-workspace.
Diffs
-----
src/netwm_p.h 1c26d47257a4f3ce71c360c394cc5be16f32b18c
src/netwm.h a5ad46fa1f0255ca6719224079bdea4598b48c51
src/netwm.cpp 288a34348f464e1158640afa1d931d9d05690cc4
src/kwindowsystem_x11.cpp 95c396b65ae0a24db6a276b9b72f4175bb7c14cc
autotests/netrootinfotestwm.cpp 120fbee92d0b22862d8ce746b3b30891ecd9f056
Diff: https://git.reviewboard.kde.org/r/116610/diff/
Testing
-------
tests succeed with 116609 applied. KWin and kde-workspace will get adjusted to test.
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140317/0b6be4ec/attachment.html>
More information about the Kde-frameworks-devel
mailing list