shithub: libvpx

Info  •  Files  •  Log  •  Branches

Refactor block_rd_txfm()

Merge two identical if branches.

Change-Id: Ie012ba9c116a30ef6fa2e7868c7a4ba886b99bc6

--- a/vp9/encoder/vp9_rdopt.c

+++ b/vp9/encoder/vp9_rdopt.c

@@ -701,9 +701,11 @@

                       blk_row, blk_col, plane_bsize, tx_bsize);

       dist = (int64_t)tmp * 16;

     }

-  } else if (max_txsize_lookup[plane_bsize] == tx_size) {

-    if (x->skip_txfm[(plane << 2) + (block >> (tx_size << 1))] ==

-        SKIP_TXFM_NONE) {

+  } else {

+    int skip_txfm_flag = SKIP_TXFM_NONE;

+    if (max_txsize_lookup[plane_bsize] == tx_size)

+      skip_txfm_flag = x->skip_txfm[(plane << 2) + (block >> (tx_size << 1))];

+    if (skip_txfm_flag == SKIP_TXFM_NONE) {

       // full forward transform and quantization

       vp9_xform_quant(x, plane, block, blk_row, blk_col, plane_bsize, tx_size);

       if (x->block_qcoeff_opt)

@@ -710,8 +712,7 @@

         vp9_optimize_b(x, plane, block, tx_size, coeff_ctx);

       dist_block(args->cpi, x, plane, plane_bsize, block, blk_row, blk_col,

                  tx_size, &dist, &sse);

-    } else if (x->skip_txfm[(plane << 2) + (block >> (tx_size << 1))] ==

-               SKIP_TXFM_AC_ONLY) {

+    } else if (skip_txfm_flag == SKIP_TXFM_AC_ONLY) {

       // compute DC coefficient

       tran_low_t *const coeff = BLOCK_OFFSET(x->plane[plane].coeff, block);

       tran_low_t *const dqcoeff = BLOCK_OFFSET(xd->plane[plane].dqcoeff, block);

@@ -738,13 +739,6 @@

       sse = x->bsse[(plane << 2) + (block >> (tx_size << 1))] << 4;

       dist = sse;

     }

-  } else {

-    // full forward transform and quantization

-    vp9_xform_quant(x, plane, block, blk_row, blk_col, plane_bsize, tx_size);

-    if (x->block_qcoeff_opt)

-      vp9_optimize_b(x, plane, block, tx_size, coeff_ctx);

-    dist_block(args->cpi, x, plane, plane_bsize, block, blk_row, blk_col,

-               tx_size, &dist, &sse);

   }

   rd = RDCOST(x->rdmult, x->rddiv, 0, dist);