Review Request 112755: Reimplement KXUtils::createPixmapFromHandle with XCB
Fredrik Höglund
fredrik at kde.org
Mon Sep 23 13:37:09 UTC 2013
> On Sept. 23, 2013, 10:27 a.m., Kevin Ottens wrote:
> > Not that I'm really qualified with xcb code, but it looks ok to me.
>
> Martin Gräßlin wrote:
> no, no. The issues mentioned by Fredrik should be fixed.
Kevin, please don't take this the wrong way, but it would be enormously helpful if you didn't approve patches
you yourself claim to NOT be qualified to review, when others who ARE qualified have already reviewed them and
voiced concerns.
This is not the first time you have done this either.
- Fredrik
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112755/#review40515
-----------------------------------------------------------
On Sept. 17, 2013, 5:42 a.m., Martin Gräßlin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112755/
> -----------------------------------------------------------
>
> (Updated Sept. 17, 2013, 5:42 a.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Description
> -------
>
> Implements the createPixmapFromHandle by getting the image for the pixmaps and using it as either the Pixmap or the bitmap mask.
>
>
> Diffs
> -----
>
> tier1/kwindowsystem/src/kxutils.cpp 33bd678
> tier1/kwindowsystem/src/kxutils_p.h 84d639b
>
> Diff: http://git.reviewboard.kde.org/r/112755/diff/
>
>
> Testing
> -------
>
> Adjusted KWin to take this codepath and say thanks to Iceweasel for having a mask
>
>
> Thanks,
>
> Martin Gräßlin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130923/36e3cd40/attachment.html>
More information about the Kde-frameworks-devel
mailing list