commit 03467d0e9854c25f9170d84c4d689b5c42364fe8
author Hui Su <huisu@google.com> Tue Apr 16 11:34:35 2019 -0700
committer Hui Su <huisu@google.com> Tue Apr 16 21:30:58 2019 +0000
tree 44ee9131ce0f38e3e90bde101028dffc12d9053a
parent e10416df42b264d5588679259ce0deb066583461

Cleanup rd_test_partition3()

Change-Id: I56b0c02d80da2d3006f39a321a99472a6abe94d0

av1/encoder/encodeframe.c

diff --git a/av1/encoder/encodeframe.c b/av1/encoder/encodeframe.c
index e629b27..b2b4a6a 100644
--- a/av1/encoder/encodeframe.c
+++ b/av1/encoder/encodeframe.c
@@ -2139,46 +2139,33 @@
                                int mi_row0, int mi_col0, BLOCK_SIZE subsize0,
                                int mi_row1, int mi_col1, BLOCK_SIZE subsize1,
                                int mi_row2, int mi_col2, BLOCK_SIZE subsize2) {
-  MACROBLOCK *const x = &td->mb;
-  MACROBLOCKD *const xd = &x->e_mbd;
+  const MACROBLOCK *const x = &td->mb;
+  const MACROBLOCKD *const xd = &x->e_mbd;
+  const int pl = partition_plane_context(xd, mi_row, mi_col, bsize);
   RD_STATS sum_rdc;
-#define RTP_STX_TRY_ARGS
-  int pl = partition_plane_context(xd, mi_row, mi_col, bsize);
   av1_init_rd_stats(&sum_rdc);
   sum_rdc.rate = x->partition_cost[pl][partition];
   sum_rdc.rdcost = RDCOST(x->rdmult, sum_rdc.rate, 0);
   if (!rd_try_subblock(cpi, td, tile_data, tp, 0, mi_row0, mi_col0, subsize0,
-                       best_rdc->rdcost, &sum_rdc, RTP_STX_TRY_ARGS partition,
-                       ctx, &ctxs[0]))
+                       best_rdc->rdcost, &sum_rdc, partition, ctx, &ctxs[0]))
     return;
 
   if (!rd_try_subblock(cpi, td, tile_data, tp, 0, mi_row1, mi_col1, subsize1,
-                       best_rdc->rdcost, &sum_rdc, RTP_STX_TRY_ARGS partition,
-                       &ctxs[0], &ctxs[1]))
+                       best_rdc->rdcost, &sum_rdc, partition, &ctxs[0],
+                       &ctxs[1]))
     return;
 
-  // With the new layout of mixed partitions for PARTITION_HORZ_B and
-  // PARTITION_VERT_B, the last subblock might start past halfway through the
-  // main block, so we might signal it even though the subblock lies strictly
-  // outside the image. In that case, we won't spend any bits coding it and the
-  // difference (obviously) doesn't contribute to the error.
-  const int try_block2 = 1;
-  if (try_block2 &&
-      !rd_try_subblock(cpi, td, tile_data, tp, 1, mi_row2, mi_col2, subsize2,
-                       best_rdc->rdcost, &sum_rdc, RTP_STX_TRY_ARGS partition,
-                       &ctxs[1], &ctxs[2]))
+  if (!rd_try_subblock(cpi, td, tile_data, tp, 1, mi_row2, mi_col2, subsize2,
+                       best_rdc->rdcost, &sum_rdc, partition, &ctxs[1],
+                       &ctxs[2]))
     return;
 
   if (sum_rdc.rdcost >= best_rdc->rdcost) return;
-
   sum_rdc.rdcost = RDCOST(x->rdmult, sum_rdc.rate, sum_rdc.dist);
-
   if (sum_rdc.rdcost >= best_rdc->rdcost) return;
 
   *best_rdc = sum_rdc;
   pc_tree->partitioning = partition;
-
-#undef RTP_STX_TRY_ARGS
 }
 
 static void reset_partition(PC_TREE *pc_tree, BLOCK_SIZE bsize) {