Skip to content
Snippets Groups Projects
Commit 9699b3a2 authored by Alberto Delmás's avatar Alberto Delmás Committed by Kostya Shishkov
Browse files

mss12: avoid unnecessary division in arith*_get_bit()


That division can be replaced with a comparison:
((c->value - c->low) << 1) + 1 >= range

By expanding 'range' definition and simplifying this inequation we obtain
the final expression.

Suggested by Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: default avatarKostya Shishkov <kostya.shishkov@gmail.com>
parent 6ceef07b
No related branches found
No related tags found
No related merge requests found
...@@ -103,7 +103,7 @@ av_cold int ff_mss12_decode_end(MSS12Context *ctx); ...@@ -103,7 +103,7 @@ av_cold int ff_mss12_decode_end(MSS12Context *ctx);
static int arith ## VERSION ## _get_bit(ArithCoder *c) \ static int arith ## VERSION ## _get_bit(ArithCoder *c) \
{ \ { \
int range = c->high - c->low + 1; \ int range = c->high - c->low + 1; \
int bit = (((c->value - c->low) << 1) + 1) / range; \ int bit = 2 * c->value - c->low >= c->high; \
\ \
if (bit) \ if (bit) \
c->low += range >> 1; \ c->low += range >> 1; \
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment