]> granicus.if.org Git - libvpx/commit
Bug fix in probability update savings computation
authorDeb Mukherjee <debargha@google.com>
Wed, 28 Mar 2012 22:19:45 +0000 (15:19 -0700)
committerDeb Mukherjee <debargha@google.com>
Thu, 29 Mar 2012 15:39:02 +0000 (08:39 -0700)
commit78ecbc98e4df70e94e8f2029c8d2d65f643e6f74
tree13e825a4e34475441f9cff49a54cbeb47c3666b4
parent9f900a1c6404464edd23d2d43a3ab1f390ec8af7
Bug fix in probability update savings computation

Found this bug while tracking down some anomalies in my experiments.
Since vp8_cost_one and vp8_cost_zero return unsigned int, the
bit shift by 8 will be incorrect if the value is negative.
I am cautiously optimistic that this fix will make the prob
updates more correct and somewhat improve results across the board.
But the update probabilities will need to be retuned I think.

Patch 2: Adding more of the same fixes using a macro.

Change-Id: I1a168f040e74e8c67e7225103b1c2af9a611da49
vp8/encoder/bitstream.c