Skip to content

Commit a51116c

Browse files
committed
test: fix accidental ub in rand_dist test
1 parent 2c1e34e commit a51116c

1 file changed

Lines changed: 5 additions & 10 deletions

File tree

test/t-rand_dist-17.cpp

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -257,22 +257,16 @@ TEST_CASE("std compat") {
257257
}
258258
}
259259

260-
#define uauto [[maybe_unused]] auto
261-
262260
TEST_CASE("degenerate") {
263261
SUBCASE("int") {
264262
q_test_rng<uint32_t> rng; // empty rng, should not be used
265263
SUBCASE("0 range") {
266264
itlib::uniform_uint_max_distribution<uint32_t> dist(0);
267-
for (uauto _ : rng.values) {
268-
CHECK(dist(rng) == 0);
269-
}
265+
CHECK(dist(rng) == 0);
270266
}
271267
SUBCASE("fixed range") {
272268
itlib::uniform_int_distribution<int32_t> dist(42, 42);
273-
for (uauto _ : rng.values) {
274-
CHECK(dist(rng) == 42);
275-
}
269+
CHECK(dist(rng) == 42);
276270
}
277271
}
278272
SUBCASE("real") {
@@ -284,15 +278,16 @@ TEST_CASE("degenerate") {
284278
(1u << 24) - 1, // almost 1
285279
1u << 23,
286280
};
281+
const auto size = int(rng.values.size());
287282
SUBCASE("0 range") {
288283
itlib::fast_uniform_real_distribution<float> dist(0, 0);
289-
for (uauto _ : rng.values) {
284+
for (int i = 0; i < size; ++i) {
290285
CHECK(dist(rng) == 0);
291286
}
292287
}
293288
SUBCASE("fixed range") {
294289
itlib::fast_uniform_real_distribution<float> dist(3.14f, 3.14f);
295-
for (uauto _ : rng.values) {
290+
for (int i = 0; i < size; ++i) {
296291
CHECK(dist(rng) == 3.14f);
297292
}
298293
}

0 commit comments

Comments
 (0)