diff options
author | Michael Niedermayer <michael@niedermayer.cc> | 2017-03-24 19:45:56 +0300 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2017-03-24 20:18:57 +0300 |
commit | d92ad42fb38a312c86b71b0e17fe860f0c881be4 (patch) | |
tree | 35ee14c2c12c80f553e5b4f8d2546094fdc60a53 /libavcodec/tests | |
parent | 824fe914fee7fb5dc8c8c14940e511cda089c36d (diff) |
avcodec/tests/celp_math: Change IsAlmostEqual() to a function
Fixes empty statement, found by ubitux
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec/tests')
-rw-r--r-- | libavcodec/tests/celp_math.c | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/libavcodec/tests/celp_math.c b/libavcodec/tests/celp_math.c index 4b1c1867b5..669ea70362 100644 --- a/libavcodec/tests/celp_math.c +++ b/libavcodec/tests/celp_math.c @@ -19,6 +19,15 @@ #include "libavutil/libm.h" #include "libavcodec/celp_math.c" +static inline void IsAlmostEqual(float A, float B, float epsilon) +{ + float diff = fabsf(A - B); + float absa = fabsf(A); + float absb = fabsf(B); + float largest = (absb > absa) ? absb : absa; + av_assert0(diff <= largest * epsilon); +} + int main(void) { int i; @@ -26,18 +35,10 @@ int main(void) const float f2[3] = {3.3, 4.4, 5.5}; const int16_t i1[3] = {6, 7, 8}; const int16_t i2[3] = {9, 10, 11}; - float diff, largest, absa, absb; float r = ff_dot_productf(f1, f2, FF_ARRAY_ELEMS(f1)); int64_t d = ff_dot_product(i1, i2, FF_ARRAY_ELEMS(i1)); -# define IsAlmostEqual(A, B, epsilon) \ - diff = fabsf(A - B); \ - absa = fabsf(A); \ - absb = fabsf(B); \ - largest = (absb > absa) ? absb : absa; \ - av_assert0(diff <= largest * epsilon); - IsAlmostEqual(16.94f, r, 0.000001f); av_assert0(212 == d); |