]> granicus.if.org Git - libvpx/commitdiff
Cleanup dist_block()
authorAlex Converse <aconverse@google.com>
Thu, 16 Jun 2016 22:27:22 +0000 (15:27 -0700)
committerAlex Converse <aconverse@google.com>
Thu, 16 Jun 2016 22:27:22 +0000 (15:27 -0700)
Change-Id: Iff0c0548924efd5a01c3a301cc5b4cdfda42e87e

vp10/encoder/rdopt.c

index b000da827dc79383c2c4d524dd9ba588c7aca1c6..c13f89a11f0f9e85b2ed4007a1342aa3b4451705 100644 (file)
@@ -1093,13 +1093,13 @@ static int cost_coeffs(MACROBLOCK *x,
 static void dist_block(const VP10_COMP *cpi, MACROBLOCK *x, int plane,
                        int block, int blk_row, int blk_col, TX_SIZE tx_size,
                        int64_t *out_dist, int64_t *out_sse) {
+  MACROBLOCKD* const xd = &x->e_mbd;
+  const struct macroblock_plane *const p = &x->plane[plane];
+  const struct macroblockd_plane *const pd = &xd->plane[plane];
   if (cpi->sf.use_transform_domain_distortion) {
     // Transform domain distortion computation is more accurate as it does
     // not involve an inverse transform, but it is less accurate.
     const int ss_txfrm_size = tx_size << 1;
-    MACROBLOCKD* const xd = &x->e_mbd;
-    const struct macroblock_plane *const p = &x->plane[plane];
-    const struct macroblockd_plane *const pd = &xd->plane[plane];
     int64_t this_sse;
     int tx_type = get_tx_type(pd->plane_type, xd, block, tx_size);
     int shift = (MAX_TX_SCALE - get_tx_scale(xd, tx_type, tx_size)) * 2;
@@ -1120,11 +1120,6 @@ static void dist_block(const VP10_COMP *cpi, MACROBLOCK *x, int plane,
   } else {
     const BLOCK_SIZE tx_bsize = txsize_to_bsize[tx_size];
     const int bs = 4*num_4x4_blocks_wide_lookup[tx_bsize];
-
-    const MACROBLOCKD *xd = &x->e_mbd;
-    const struct macroblock_plane *p = &x->plane[plane];
-    const struct macroblockd_plane *pd = &xd->plane[plane];
-
     const int src_stride = x->plane[plane].src.stride;
     const int dst_stride = xd->plane[plane].dst.stride;
     const int src_idx = 4 * (blk_row * src_stride + blk_col);
@@ -1132,7 +1127,7 @@ static void dist_block(const VP10_COMP *cpi, MACROBLOCK *x, int plane,
     const uint8_t *src = &x->plane[plane].src.buf[src_idx];
     const uint8_t *dst = &xd->plane[plane].dst.buf[dst_idx];
     const tran_low_t *dqcoeff = BLOCK_OFFSET(pd->dqcoeff, block);
-    const uint16_t *eob = &p->eobs[block];
+    const uint16_t eob = p->eobs[block];
 
     unsigned int tmp;
 
@@ -1141,7 +1136,7 @@ static void dist_block(const VP10_COMP *cpi, MACROBLOCK *x, int plane,
     cpi->fn_ptr[tx_bsize].vf(src, src_stride, dst, dst_stride, &tmp);
     *out_sse = (int64_t)tmp * 16;
 
-    if (*eob) {
+    if (eob) {
       const MB_MODE_INFO *mbmi = &xd->mi[0]->mbmi;
 #if CONFIG_VP9_HIGHBITDEPTH
       DECLARE_ALIGNED(16, uint16_t, recon16[MAX_TX_SQUARE]);
@@ -1156,7 +1151,7 @@ static void dist_block(const VP10_COMP *cpi, MACROBLOCK *x, int plane,
 
       inv_txfm_param.tx_type = get_tx_type(plane_type, xd, block, tx_size);
       inv_txfm_param.tx_size = tx_size;
-      inv_txfm_param.eob = *eob;
+      inv_txfm_param.eob = eob;
       inv_txfm_param.lossless = xd->lossless[mbmi->segment_id];
 
 #if CONFIG_VP9_HIGHBITDEPTH