commit 3b93e8eb0b3cc9bc11e172d614d9c5bcfc9902fe
author Thomas Davies <thdavies@cisco.com> Wed Sep 20 09:59:07 2017 +0100
committer Sebastien Alaiwan <sebastien.alaiwan@allegrodvt.com> Tue Sep 26 08:36:30 2017 +0000
tree b4f36e8575375af91ac68893e694a0457ee5eb90
parent 5589d71ccc1fb0bafd404680a3d1616d19afc8a1

Fix redundancy in delta_q signalling.

rem_bits-1 not rem_bits should be transmitted so that
2 cannot be coded in two ways.

 BUG=aomedia:811

Change-Id: Iaa0203214bbe6fc6775e05fe4b6e976d568f54b3

av1/encoder/bitstream.c

diff --git a/av1/encoder/bitstream.c b/av1/encoder/bitstream.c
index 7db0afc..a3879d4 100644
--- a/av1/encoder/bitstream.c
+++ b/av1/encoder/bitstream.c
@@ -533,7 +533,7 @@
   if (!smallval) {
     rem_bits = OD_ILOG_NZ(abs - 1) - 1;
     thr = (1 << rem_bits) + 1;
-    aom_write_literal(w, rem_bits, 3);
+    aom_write_literal(w, rem_bits - 1, 3);
     aom_write_literal(w, abs - thr, rem_bits);
   }
   if (abs > 0) {
@@ -557,7 +557,7 @@
   if (!smallval) {
     rem_bits = OD_ILOG_NZ(abs - 1) - 1;
     thr = (1 << rem_bits) + 1;
-    aom_write_literal(w, rem_bits, 3);
+    aom_write_literal(w, rem_bits - 1, 3);
     aom_write_literal(w, abs - thr, rem_bits);
   }
   if (abs > 0) {