[FFmpeg-devel] [PATCH] vp56: remove clear_blocks call, and clear alpha plane U/V DC only.

Ronald S. Bultje rsbultje at gmail.com
Fri Feb 1 20:04:50 CET 2013


From: "Ronald S. Bultje" <rsbultje at gmail.com>

The non-alpha and alpha-Y planes are cleared in the idct_put/add()
calls. For the alpha U/V planes, we only care about the DC for entropy
context prediction purposes, the rest of the data is unused.
---
 libavcodec/vp56.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c
index 7df5ada..6cc3770 100644
--- a/libavcodec/vp56.c
+++ b/libavcodec/vp56.c
@@ -394,8 +394,6 @@ static void vp56_decode_mb(VP56Context *s, int row, int col, int is_alpha)
         mb_type = vp56_decode_mv(s, row, col);
     ref_frame = vp56_reference_frame[mb_type];
 
-    s->dsp.clear_blocks(*s->block_coeff);
-
     s->parse_coeff(s);
 
     vp56_add_predictors_dc(s, ref_frame);
@@ -448,6 +446,11 @@ static void vp56_decode_mb(VP56Context *s, int row, int col, int is_alpha)
             }
             break;
     }
+
+    if (is_alpha) {
+        s->block_coeff[4][0] = 0;
+        s->block_coeff[5][0] = 0;
+    }
 }
 
 static int vp56_size_changed(VP56Context *s)
-- 
1.8.0



More information about the ffmpeg-devel mailing list