Make sure 32x32 coefficients fit in int16_t.
authorRonald S. Bultje <rbultje@google.com>
Tue, 30 Apr 2013 00:38:25 +0000 (17:38 -0700)
committerRonald S. Bultje <rbultje@google.com>
Tue, 30 Apr 2013 00:59:00 +0000 (17:59 -0700)
Fixes mismatches on yt.

Change-Id: I65444137578b29a4984af5c01b2874c5ab086acc

vp9/decoder/vp9_detokenize.c
vp9/decoder/vp9_idct_blk.c

index 3ef1e5b..650defd 100644 (file)
@@ -85,7 +85,8 @@ DECLARE_ALIGNED(16, extern const uint8_t, vp9_norm[256]);
 
 #define WRITE_COEF_CONTINUE(val, token)                  \
   {                                                      \
-    qcoeff_ptr[scan[c]] = vp9_read_and_apply_sign(r, val) * dq[c > 0]; \
+    qcoeff_ptr[scan[c]] = vp9_read_and_apply_sign(r, val) * \
+                            dq[c > 0] / (1 + (txfm_size == TX_32X32)); \
     INCREMENT_COUNT(token);                              \
     c++;                                                 \
     continue;                                            \
index 73c1c4d..3480df2 100644 (file)
@@ -315,23 +315,12 @@ void vp9_idct_add_32x32_c(int16_t *input, uint8_t *dest, int stride, int eob) {
   DECLARE_ALIGNED_ARRAY(16, int16_t, output, 1024);
 
   if (eob) {
-    input[0] = input[0] / 2;
     if (eob == 1) {
       vp9_short_idct1_32x32(input, output);
       vp9_add_constant_residual_32x32(output[0], dest, stride);
       input[0] = 0;
 #if !CONFIG_SCATTERSCAN
     } else if (eob <= 10) {
-      input[1] = input[1] / 2;
-      input[2] = input[2] / 2;
-      input[3] = input[3] / 2;
-      input[32] = input[32] / 2;
-      input[33] = input[33] / 2;
-      input[34] = input[34] / 2;
-      input[64] = input[64] / 2;
-      input[65] = input[65] / 2;
-      input[96] = input[96] / 2;
-
       // the idct halves ( >> 1) the pitch
       vp9_short_idct10_32x32(input, output, 64);
 
@@ -343,10 +332,6 @@ void vp9_idct_add_32x32_c(int16_t *input, uint8_t *dest, int stride, int eob) {
       vp9_add_residual_32x32(output, dest, stride);
 #endif
     } else {
-      int i;
-      for (i = 1; i < 1024; i++)
-        input[i] = input[i] / 2;
-
       vp9_short_idct32x32(input, output, 64);
       vpx_memset(input, 0, 2048);
       vp9_add_residual_32x32(output, dest, stride);