From: Steinar H. Gunderson <sgunderson@bigfoot.com>
Date: Tue, 2 Jan 2018 17:03:35 +0000 (+0100)
Subject: Make sure we do not send infinities to benchmarking textures.
X-Git-Tag: 1.6.0~13
X-Git-Url: https://git.sesse.net/?p=movit;a=commitdiff_plain;h=037ee905bc05fcbb278ac3a0b1f73249efada9b2;hp=825ae6ce6e426b11beb9ba2089b2c440e7ed94b2

Make sure we do not send infinities to benchmarking textures.
---

diff --git a/deinterlace_effect_test.cpp b/deinterlace_effect_test.cpp
index dec1322..a509f8c 100644
--- a/deinterlace_effect_test.cpp
+++ b/deinterlace_effect_test.cpp
@@ -243,11 +243,11 @@ void BM_DeinterlaceEffect(benchmark::State &state, TestFormat format, bool spati
 	unique_ptr<float[]> out_data(new float[width * height * format.bytes_per_pixel]);
 
 	for (unsigned i = 0; i < width * field_height * format.bytes_per_pixel; ++i) {
-		field1[i] = rand();
-		field2[i] = rand();
-		field3[i] = rand();
-		field4[i] = rand();
-		field5[i] = rand();
+		field1[i] = rand() / (RAND_MAX + 1.0);
+		field2[i] = rand() / (RAND_MAX + 1.0);
+		field3[i] = rand() / (RAND_MAX + 1.0);
+		field4[i] = rand() / (RAND_MAX + 1.0);
+		field5[i] = rand() / (RAND_MAX + 1.0);
 	}
 
 	EffectChainTester tester(nullptr, width, height);
diff --git a/resample_effect_test.cpp b/resample_effect_test.cpp
index 4faa810..71354f2 100644
--- a/resample_effect_test.cpp
+++ b/resample_effect_test.cpp
@@ -452,7 +452,7 @@ void BM_ResampleEffect(benchmark::State &state, GammaCurve gamma_curve, GLenum o
 	unique_ptr<T[]> out_data(new T[out_width * out_height * 4]);
 
 	for (unsigned i = 0; i < in_width * in_height * 4; ++i) {
-		data[i] = from_fp32<T>(float(rand()));
+		data[i] = from_fp32<T>(rand() / (RAND_MAX + 1.0));
 	}
 
 	EffectChainTester tester(nullptr, out_width, out_height, FORMAT_BGRA_POSTMULTIPLIED_ALPHA, COLORSPACE_sRGB, gamma_curve, output_format);