Review Request 112755: Reimplement KXUtils::createPixmapFromHandle with XCB

Fredrik Höglund fredrik at kde.org
Mon Sep 16 12:16:07 UTC 2013


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



tier1/kwindowsystem/src/kxutils.cpp
<http://git.reviewboard.kde.org/r/112755/#comment29677>

    X pixmaps use premultiplied alpha.



tier1/kwindowsystem/src/kxutils.cpp
<http://git.reviewboard.kde.org/r/112755/#comment29678>

    A bitmap is typically fetched as an XY_PIXMAP.
    



tier1/kwindowsystem/src/kxutils_p.h
<http://git.reviewboard.kde.org/r/112755/#comment29676>

    This statement is no longer true. If the pixmap is not an ARGB32 pixmap, the function will crash. It will also fail if the image byte order doesn't match the CPU byte order, if the mask has the wrong size, or the mask isn't a bitmap.
    


- Fredrik Höglund


On Sept. 16, 2013, 7:27 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. 16, 2013, 7:27 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/20130916/90bb4621/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list