commit f71157fa06f765045365d3c22fb346f5a0a1fe02
author Sebastian Dröge <sebastian.droege@collabora.co.uk> Thu Apr 22 15:45:15 2010 +0200
committer Sebastian Dröge <sebastian.droege@collabora.co.uk> Thu Apr 29 19:28:22 2010 +0200
tree 0960cd92a60c74c32bf9521dbc8ee8711d11099d
parent 1897ab2928d7017e3c210c7b0dfe534e67b2d227

alpha: Fix planar YUV->RGB processing

gst/alpha/gstalpha.c

diff --git a/gst/alpha/gstalpha.c b/gst/alpha/gstalpha.c
index 2862b94..55da9a2 100644
--- a/gst/alpha/gstalpha.c
+++ b/gst/alpha/gstalpha.c
@@ -1514,11 +1514,10 @@
     gint height, GstAlpha * alpha)
 {
   gint b_alpha = CLAMP ((gint) (alpha->alpha * 255), 0, 255);
-  const guint8 *srcY;
+  const guint8 *srcY, *srcY_tmp;
   const guint8 *srcU, *srcU_tmp;
   const guint8 *srcV, *srcV_tmp;
   gint i, j;
-  gint src_wrap, src_uv_wrap;
   gint y_stride, uv_stride;
   gint v_subs, h_subs;
   gint matrix[12];
@@ -1542,10 +1541,7 @@
   y_stride = gst_video_format_get_row_stride (alpha->in_format, 0, width);
   uv_stride = gst_video_format_get_row_stride (alpha->in_format, 1, width);
 
-  src_wrap = y_stride - width;
-  src_uv_wrap = uv_stride - (width / 2);
-
-  srcY = src;
+  srcY_tmp = srcY = src;
   srcU_tmp = srcU =
       src + gst_video_format_get_component_offset (alpha->in_format, 1, width,
       height);
@@ -1601,10 +1597,10 @@
       }
     }
 
-    srcY += src_wrap;
+    srcY_tmp = srcY = srcY_tmp + y_stride;
     if ((i + 1) % v_subs == 0) {
-      srcU_tmp = srcU += src_uv_wrap;
-      srcV_tmp = srcV += src_uv_wrap;
+      srcU_tmp = srcU = srcU_tmp + uv_stride;
+      srcV_tmp = srcV = srcV_tmp + uv_stride;
     } else {
       srcU = srcU_tmp;
       srcV = srcV_tmp;
@@ -1617,13 +1613,12 @@
     gint width, gint height, GstAlpha * alpha)
 {
   gint b_alpha = CLAMP ((gint) (alpha->alpha * 255), 0, 255);
-  const guint8 *srcY;
+  const guint8 *srcY, *srcY_tmp;
   const guint8 *srcU, *srcU_tmp;
   const guint8 *srcV, *srcV_tmp;
   gint i, j;
   gint a, y, u, v;
   gint r, g, b;
-  gint src_wrap, src_uv_wrap;
   gint y_stride, uv_stride;
   gint v_subs, h_subs;
   gint smin = 128 - alpha->black_sensitivity;
@@ -1654,10 +1649,7 @@
   y_stride = gst_video_format_get_row_stride (alpha->in_format, 0, width);
   uv_stride = gst_video_format_get_row_stride (alpha->in_format, 1, width);
 
-  src_wrap = y_stride - width;
-  src_uv_wrap = uv_stride - (width / 2);
-
-  srcY = src;
+  srcY_tmp = srcY = src;
   srcU_tmp = srcU =
       src + gst_video_format_get_component_offset (alpha->in_format, 1, width,
       height);
@@ -1720,10 +1712,10 @@
       }
     }
 
-    srcY += src_wrap;
+    srcY_tmp = srcY = srcY_tmp + y_stride;
     if ((i + 1) % v_subs == 0) {
-      srcU_tmp = srcU += src_uv_wrap;
-      srcV_tmp = srcV += src_uv_wrap;
+      srcU_tmp = srcU = srcU_tmp + uv_stride;
+      srcV_tmp = srcV = srcV_tmp + uv_stride;
     } else {
       srcU = srcU_tmp;
       srcV = srcV_tmp;