After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 729742 - glcolorconvert: Should set alpha to 1 if source is opaque
glcolorconvert: Should set alpha to 1 if source is opaque
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.3.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-05-07 19:19 UTC by Nicolas Dufresne (ndufresne)
Modified: 2014-05-12 12:27 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Nicolas Dufresne (ndufresne) 2014-05-07 19:19:28 UTC
Currently we simply assume that BGRx is like BGRA, though in first form the x means that the padding value is not guarantied opaque. For this reason, glcolorconvert should take this into account, and explicitly set the alpha to 1.0 rather then using the padding value.

On armsoc X11 driver, all windows have ARGB visual, which lead to randomly transparent window with BGRx.
Comment 1 Sebastian Dröge (slomo) 2014-05-07 19:24:37 UTC
We should never ever use the x in the RGBx variants for anything, and instead take 1 or whatever value is correct.
Comment 2 Matthew Waters (ystreet00) 2014-05-12 12:15:18 UTC
commit bd3e6007101fca3016a870599f7d7c1ac0219857
Author: Matthew Waters <ystreet00@gmail.com>
Date:   Mon May 12 21:56:06 2014 +1000

    gl/convert: clobber alpha channel when needed
    
    Needed in the conversion from padded RGB formats to formats with an
    alpha channel.  e.g. BGRx -> ABGR, etc
    
    https://bugzilla.gnome.org/show_bug.cgi?id=729742