Review Request 111792: XCB event handling for KWindowSystem

Martin Gräßlin mgraesslin at kde.org
Fri Aug 2 08:26:09 UTC 2013



> On Aug. 2, 2013, 10:18 a.m., Kevin Ottens wrote:
> > I don't feel like blocking this patch you know much more than me on this topic and I didn't spot anything obviously wrong. :-)
> > 
> > There's a few style issues in the patch (mainly space in between parenthesis), but since this whole module is rather bad for following the style it's minor and will get fixed later.

> There's a few style issues in the patch
I tried to fix as many as possible. At least I run kdevelop's reformat source on the changed code paths. But I agree the whole module needs a proper automatic cleanup.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111792/#review36954
-----------------------------------------------------------


On July 30, 2013, 10:34 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111792/
> -----------------------------------------------------------
> 
> (Updated July 30, 2013, 10:34 a.m.)
> 
> 
> 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/a2a74169/attachment.html>


More information about the Kde-frameworks-devel mailing list