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 727188 - ximagesink: Should set alpha_mask to 0 when dealing with 24bit RGB
ximagesink: Should set alpha_mask to 0 when dealing with 24bit RGB
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 1.3.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-03-27 20:38 UTC by Nicolas Dufresne (ndufresne)
Modified: 2014-06-22 12:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[PATCH] ximagesrc: only extrapolate alpha mask for 32-bit depth (1.13 KB, patch)
2014-03-27 20:43 UTC, Nicolas Dufresne (ndufresne)
committed Details | Review

Description Nicolas Dufresne (ndufresne) 2014-03-27 20:38:28 UTC
Currently we always set the alpha_mask, even if there is no alpha. This rely on the format from mask code to ignore it, it just safer to set it to zero as it should.

Same has been done in ximagesrc, see https://bugzilla.gnome.org/show_bug.cgi?id=726833
Comment 1 Nicolas Dufresne (ndufresne) 2014-03-27 20:43:54 UTC
Created attachment 273113 [details] [review]
[PATCH] ximagesrc: only extrapolate alpha mask for 32-bit depth


Instead of passing bogus alpha mask values when there's no alpha.

https://bugzilla.gnome.org/show_bug.cgi?id=727188
---
 sys/ximage/ximagesink.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
Comment 2 Nicolas Dufresne (ndufresne) 2014-03-27 20:48:50 UTC
(In reply to comment #1)
> [PATCH] ximagesrc: only extrapolate alpha mask for 32-bit depth

Oops, ximagesink. Fix in my repo.


commit f270b267c530ccc85718b37df74a9b01f3e4bca3
Author: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Date:   Thu Mar 27 16:43:10 2014 -0400

    ximagesink: only extrapolate alpha mask for 32-bit depth
    
    Instead of passing bogus alpha mask values when there's no alpha.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=727188