commit 3e4c9ed2b2f40959fb5a7080cefde9d7c9be8801
author Jingning Han <jingning@google.com> Fri Jun 07 15:03:16 2019 -0700
committer Jingning Han <jingning@google.com> Mon Jun 10 05:27:28 2019 +0000
tree 36adf29aa3b32ce32ff547e3d57b51d4003326fb
parent 4e585cccdbd74db65b83f5281ed37e62d3c05fb0

Add assert to clear static analysis warning in optimize_txb

The optimize_txb function is not called when eob is 0. Add assert
to clear the static analysis warning triggered by next change that
enables it to support qmatrix.

BUG=aomedia:2312

Change-Id: I8dfa9236fdb65bf4c85a69c7acdc2c0338ca8214

av1/encoder/encodetxb.c

diff --git a/av1/encoder/encodetxb.c b/av1/encoder/encodetxb.c
index a70ebc8..f8975e4 100644
--- a/av1/encoder/encodetxb.c
+++ b/av1/encoder/encodetxb.c
@@ -1659,6 +1659,9 @@
   tran_low_t *dqcoeff = BLOCK_OFFSET(pd->dqcoeff, block);
   const tran_low_t *tcoeff = BLOCK_OFFSET(p->coeff, block);
 
+  // This function is not called if eob = 0.
+  assert(eob > 0);
+
   if (fast_mode) {
     update_coeff_eob_fast(&eob, shift, dequant, scan, tcoeff, qcoeff, dqcoeff);
     p->eobs[block] = eob;