summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIlya Tumaykin <itumaykin@gmail.com>2018-02-01 14:05:06 +0300
committerKevin Mitchell <kevmitch@gmail.com>2018-02-03 13:56:08 -0800
commitf4f24c105f9c132e84cba9a9707acc261033a816 (patch)
tree6a0b1ffbd5a16acdbcfa70ee80aaecd527b0d765
parentd7db42d27f0a474a0ef0de8a771d3e4d5344e3ed (diff)
downloadmpv-f4f24c105f9c132e84cba9a9707acc261033a816.tar.bz2
mpv-f4f24c105f9c132e84cba9a9707acc261033a816.tar.xz
tests: stop comparing floats against DBL_EPSILON, use FLT_EPSILON
Fixes #5253.
-rw-r--r--test/gl_video.c10
-rw-r--r--test/test_helpers.h3
2 files changed, 7 insertions, 6 deletions
diff --git a/test/gl_video.c b/test/gl_video.c
index a2d2577e25..6b5f3a7060 100644
--- a/test/gl_video.c
+++ b/test/gl_video.c
@@ -4,22 +4,22 @@
static void test_scale_ambient_lux_limits(void **state) {
float x;
x = gl_video_scale_ambient_lux(16.0, 64.0, 2.40, 1.961, 16.0);
- assert_double_equal(x, 2.40f);
+ assert_float_equal(x, 2.40f);
x = gl_video_scale_ambient_lux(16.0, 64.0, 2.40, 1.961, 64.0);
- assert_double_equal(x, 1.961f);
+ assert_float_equal(x, 1.961f);
}
static void test_scale_ambient_lux_sign(void **state) {
float x;
x = gl_video_scale_ambient_lux(16.0, 64.0, 1.961, 2.40, 64.0);
- assert_double_equal(x, 2.40f);
+ assert_float_equal(x, 2.40f);
}
static void test_scale_ambient_lux_clamping(void **state) {
float x;
x = gl_video_scale_ambient_lux(16.0, 64.0, 2.40, 1.961, 0.0);
- assert_double_equal(x, 2.40f);
+ assert_float_equal(x, 2.40f);
}
static void test_scale_ambient_lux_log10_midpoint(void **state) {
@@ -27,7 +27,7 @@ static void test_scale_ambient_lux_log10_midpoint(void **state) {
// 32 corresponds to the the midpoint after converting lux to the log10 scale
x = gl_video_scale_ambient_lux(16.0, 64.0, 2.40, 1.961, 32.0);
float mid_gamma = (2.40 - 1.961) / 2 + 1.961;
- assert_double_equal(x, mid_gamma);
+ assert_float_equal(x, mid_gamma);
}
int main(void) {
diff --git a/test/test_helpers.h b/test/test_helpers.h
index 7a61da82ea..49328f623f 100644
--- a/test/test_helpers.h
+++ b/test/test_helpers.h
@@ -10,6 +10,7 @@
#include <math.h>
#include <float.h>
-#define assert_double_equal(a, b) assert_true(fabs(a - b) <= DBL_EPSILON)
+#define assert_double_equal(a, b) assert_true(fabs((a) - (b)) <= DBL_EPSILON * fmax(fabs(a), fabs(b)))
+#define assert_float_equal(a, b) assert_true(fabsf((a) - (b)) <= FLT_EPSILON * fmaxf(fabsf(a), fabsf(b)))
#endif