]> granicus.if.org Git - libvpx/commitdiff
revise two function definitions with less parameters
authorYaowu Xu <yaowu@google.com>
Fri, 20 May 2011 01:37:13 +0000 (18:37 -0700)
committerYaowu Xu <yaowu@google.com>
Fri, 20 May 2011 02:06:03 +0000 (19:06 -0700)
Change-Id: Ia96e5bf915e4d3c0ac9c1795114bd9e5dd07327a

vp8/encoder/encodeintra.c
vp8/encoder/encodeintra.h
vp8/encoder/firstpass.c
vp8/encoder/pickinter.c

index 6f7e66371bf05c35ccad3d3302172459454e06d8..307064153a4127e5ce9a1fcf07ec6f30fccccc4e 100644 (file)
 #else
 #define IF_RTCD(x) NULL
 #endif
-void vp8_encode_intra4x4block(const VP8_ENCODER_RTCD *rtcd, MACROBLOCK *x, BLOCK *be, BLOCKD *b, int best_mode)
+
+
+void vp8_encode_intra4x4block(const VP8_ENCODER_RTCD *rtcd,
+                              MACROBLOCK *x, int ib)
 {
+    BLOCKD *b = &x->e_mbd.block[ib];
+    BLOCK *be = &x->block[ib];
+
     RECON_INVOKE(&rtcd->common->recon, intra4x4_predict)
-                 (b, best_mode, b->predictor);
+                (b, b->bmi.mode, b->predictor);
 
     ENCODEMB_INVOKE(&rtcd->encodemb, subb)(be, b, 16);
 
@@ -51,13 +57,7 @@ void vp8_encode_intra4x4mby(const VP8_ENCODER_RTCD *rtcd, MACROBLOCK *mb)
     vp8_intra_prediction_down_copy(x);
 
     for (i = 0; i < 16; i++)
-    {
-        BLOCK *be = &mb->block[i];
-        BLOCKD *b = &x->block[i];
-
-        vp8_encode_intra4x4block(rtcd, mb, be, b, b->bmi.mode);
-    }
-
+        vp8_encode_intra4x4block(rtcd, mb, i);
     return;
 }
 
index 8159097c6d19967498455492c1f4f053f5536ca0..5861fd1fc39e2c7612e89dc89b96c86c47f8c889 100644 (file)
@@ -16,6 +16,6 @@
 void vp8_encode_intra16x16mby(const VP8_ENCODER_RTCD *, MACROBLOCK *x);
 void vp8_encode_intra16x16mbuv(const VP8_ENCODER_RTCD *, MACROBLOCK *x);
 void vp8_encode_intra4x4mby(const VP8_ENCODER_RTCD *, MACROBLOCK *mb);
-void vp8_encode_intra4x4block(const VP8_ENCODER_RTCD *, MACROBLOCK *x, BLOCK *be, BLOCKD *b, int best_mode);
-
+void vp8_encode_intra4x4block(const VP8_ENCODER_RTCD *rtcd,
+                              MACROBLOCK *x, int ib);
 #endif
index 7334a465dd208c674002891625feec46185d5dfe..839e53a124b4c18626c82a524964d9dc0b90869d 100644 (file)
@@ -100,10 +100,8 @@ static int encode_intra(VP8_COMP *cpi, MACROBLOCK *x, int use_dc_pred)
     {
         for (i = 0; i < 16; i++)
         {
-            BLOCKD *b = &x->e_mbd.block[i];
-            BLOCK  *be = &x->block[i];
-
-            vp8_encode_intra4x4block(IF_RTCD(&cpi->rtcd), x, be, b, B_DC_PRED);
+            x->e_mbd.block[i].bmi.mode = B_DC_PRED;
+            vp8_encode_intra4x4block(IF_RTCD(&cpi->rtcd), x, i);
         }
     }
 
index c2524921db9c3729487d7fbabf9bd29da6d90ae0..7940a518e68c7774538fa45b060174e2eafc5d2e 100644 (file)
@@ -170,8 +170,7 @@ static int get_prediction_error(BLOCK *be, BLOCKD *b, const vp8_variance_rtcd_vt
 static int pick_intra4x4block(
     const VP8_ENCODER_RTCD *rtcd,
     MACROBLOCK *x,
-    BLOCK *be,
-    BLOCKD *b,
+    int ib,
     B_PREDICTION_MODE *best_mode,
     B_PREDICTION_MODE above,
     B_PREDICTION_MODE left,
@@ -179,6 +178,9 @@ static int pick_intra4x4block(
     int *bestrate,
     int *bestdistortion)
 {
+
+    BLOCKD *b = &x->e_mbd.block[ib];
+    BLOCK *be = &x->block[ib];
     B_PREDICTION_MODE mode;
     int best_rd = INT_MAX;       // 1<<30
     int rate;
@@ -214,8 +216,7 @@ static int pick_intra4x4block(
     }
 
     b->bmi.mode = (B_PREDICTION_MODE)(*best_mode);
-    vp8_encode_intra4x4block(rtcd, x, be, b, b->bmi.mode);
-
+    vp8_encode_intra4x4block(rtcd, x, ib);
     return best_rd;
 }
 
@@ -245,8 +246,7 @@ int vp8_pick_intra4x4mby_modes
         B_PREDICTION_MODE UNINITIALIZED_IS_SAFE(best_mode);
         int UNINITIALIZED_IS_SAFE(r), UNINITIALIZED_IS_SAFE(d);
 
-        pick_intra4x4block(rtcd, mb, mb->block + i, xd->block + i,
-                               &best_mode, A, L, &r, &d);
+        pick_intra4x4block(rtcd, mb, i, &best_mode, A, L, &r, &d);
 
         cost += r;
         distortion += d;