Review Request 111792: XCB event handling for KWindowSystem
Commit Hook
null at kde.org
Fri Aug 2 11:50:21 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111792/
-----------------------------------------------------------
(Updated Aug. 2, 2013, 11:50 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks.
Description
-------
Port KWindowSystemPrivate to QAbstractNativeEventFilter
Obviously it's only filtering XCB events and the event filtering code
is ported to use XCB data types.
With this change the KSystemEventFilter is no longer needed and
therefore dropped.
Port NETWinInfo::event from XEvent to xcb_generic_event_t
Straight forward port just replacing the XLib code by XCB equivalent.
The logic of looped event compaction is dropped as inspecting the event
queue is not possible on XCB.
Port NETRootInfo::event from XEvent to xcb_generic_event_t
Straight forward port just replacing the XLib functionality to XCB.
The compaction logic got dropped as it was hard disabled anyway.
Note: this change currently breaks KWindowSystem as it also needs to
be ported.
Coding style cleanup of event handling related methods
Before adjusting the code let's have a proper coding style.
Diffs
-----
tier1/kwindowsystem/src/CMakeLists.txt PRE-CREATION
tier1/kwindowsystem/src/ksystemeventfilter.cpp PRE-CREATION
tier1/kwindowsystem/src/ksystemeventfilter_p.h PRE-CREATION
tier1/kwindowsystem/src/kwindowsystem_x11.cpp PRE-CREATION
tier1/kwindowsystem/src/netwm.h PRE-CREATION
tier1/kwindowsystem/src/netwm.cpp PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111792/diff/
Testing
-------
Part of it indirectly tested through KWin on 5. Window caption gets updated and that is triggered through NETWinInfo, so I am positively optimistic that all of it works.
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130802/413c4145/attachment.html>
More information about the Kde-frameworks-devel
mailing list