Review Request 111792: XCB event handling for KWindowSystem
Martin Gräßlin
mgraesslin at kde.org
Fri Aug 2 11:44:50 UTC 2013
> On Aug. 2, 2013, 11:29 a.m., David Faure wrote:
> > Looks good to me (I wrote the native-event-filter API in Qt5), thanks for killing KSystemEventFilter -- but please add a porting note to KDE5PORTING.html about that class. It's used outside kdelibs: http://lxr.kde.org/ident?i=KSystemEventFilter
> >
done
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111792/#review36961
-----------------------------------------------------------
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/ba634005/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list