generic-library/vpx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

INLINE and reimplement get_unsigned_bits().

Browse Source 
The new implementation disagrees when the argument is equal to 2**n but
that is never called in practice and based on how it is used the new
implementation is correct in that case.

Change-Id: Ifbac4ad87d459fe6bd2fd0f400c0340f96617342
This commit is contained in:
Alex Converse 2014-02-03 12:12:01 -08:00
parent 41d3c989fd
commit ffd3d4834b
1 changed files with 3 additions and 10 deletions

13
vp9/common/vp9_common.h

@ -18,6 +18,7 @@
#include "./vpx_config.h"
#include "vpx_mem/vpx_mem.h"
#include "vpx/vpx_integer.h"
#include "vp9/common/vp9_systemdependent.h"
#ifdef __cplusplus
extern "C" {
@ -59,16 +60,8 @@ static INLINE double fclamp(double value, double low, double high) {
return value < low ? low : (value > high ? high : value);
}
static int get_unsigned_bits(unsigned int num_values) {
int cat = 0;
if (num_values <= 1)
return 0;
num_values--;
while (num_values > 0) {
cat++;
num_values >>= 1;
}
return cat;
static INLINE int get_unsigned_bits(unsigned int num_values) {
return num_values > 0 ? get_msb(num_values) + 1 : 0;
}
#if CONFIG_DEBUG