projects / platform / upstream / gstreamer.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 21abf9c)
dvbsuboverlay: Fix blending bug regarding color channel addressing.
authorMart Raudsepp <mart.raudsepp@collabora.co.uk>
Wed, 1 Dec 2010 05:26:03 +0000 (07:26 +0200)
committerEdward Hervey <edward.hervey@collabora.co.uk>
Wed, 8 Dec 2010 15:30:10 +0000 (16:30 +0100)
libdvbsub gives us ARGB ordered pixels right now, not RGBA.

gst/dvbsuboverlay/gstdvbsuboverlay.c patch | blob | history

diff --git a/gst/dvbsuboverlay/gstdvbsuboverlay.c b/gst/dvbsuboverlay/gstdvbsuboverlay.c
index 7269cdb..71b3119 100644 (file)
--- a/gst/dvbsuboverlay/gstdvbsuboverlay.c
+++ b/gst/dvbsuboverlay/gstdvbsuboverlay.c
@@ -636,10 +636,10 @@ blit_i420 (GstDVBSubOverlay * overlay, DVBSubtitles * subs, GstBuffer * buffer)
         /* FIXME: Completely wrong blending code */
         color = sub_region->pict.palette[src[0]];
         color2 = sub_region->pict.palette[src[1]];
-        alpha = 255 - (color & 0xff);
-        r = (color >> 24) & 0xff;
-        g = (color >> 16) & 0xff;
-        b = (color >> 8) & 0xff;
+        alpha = 255 - ((color >> 24) & 0xff);
+        r = (color >> 16) & 0xff;
+        g = (color >> 8) & 0xff;
+        b = color & 0xff;
 
         Y = rgb_to_y (r, g, b);
         U = rgb_to_u (r, g, b);
@@ -678,10 +678,10 @@ blit_i420 (GstDVBSubOverlay * overlay, DVBSubtitles * subs, GstBuffer * buffer)
         /* FIXME: Completely wrong blending code */
         color = sub_region->pict.palette[src[0]];
         color2 = sub_region->pict.palette[src[1]];
-        alpha = 255 - (color & 0xff);
-        r = (color >> 24) & 0xff;
-        g = (color >> 16) & 0xff;
-        b = (color >> 8) & 0xff;
+        alpha = 255 - ((color >> 24) & 0xff);
+        r = (color >> 16) & 0xff;
+        g = (color >> 8) & 0xff;
+        b = color & 0xff;
 
         Y = rgb_to_y (r, g, b);
         U = rgb_to_u (r, g, b);
@@ -719,10 +719,10 @@ blit_i420 (GstDVBSubOverlay * overlay, DVBSubtitles * subs, GstBuffer * buffer)
         /* FIXME: Completely wrong blending code */
         color = sub_region->pict.palette[src[0]];
         color2 = sub_region->pict.palette[src[1]];
-        alpha = 255 - (color & 0xff);
-        r = (color >> 24) & 0xff;
-        g = (color >> 16) & 0xff;
-        b = (color >> 8) & 0xff;
+        alpha = 255 - ((color >> 24) & 0xff);
+        r = (color >> 16) & 0xff;
+        g = (color >> 8) & 0xff;
+        b = color & 0xff;
 
         Y = rgb_to_y (r, g, b);
         U = rgb_to_u (r, g, b);
@@ -750,10 +750,10 @@ blit_i420 (GstDVBSubOverlay * overlay, DVBSubtitles * subs, GstBuffer * buffer)
         /* FIXME: Completely wrong blending code */
         color = sub_region->pict.palette[src[0]];
         color2 = sub_region->pict.palette[src[1]];
-        alpha = 255 - (color & 0xff);
-        r = (color >> 24) & 0xff;
-        g = (color >> 16) & 0xff;
-        b = (color >> 8) & 0xff;
+        alpha = 255 - ((color >> 24) & 0xff);
+        r = (color >> 16) & 0xff;
+        g = (color >> 8) & 0xff;
+        b = color & 0xff;
 
         Y = rgb_to_y (r, g, b);
         U = rgb_to_u (r, g, b);
Domain: Multimedia / Media Common;