diff --git a/source/blender/blenlib/BLI_bounds.hh b/source/blender/blenlib/BLI_bounds.hh index d20382ed500..f5a18a0ea48 100644 --- a/source/blender/blenlib/BLI_bounds.hh +++ b/source/blender/blenlib/BLI_bounds.hh @@ -28,10 +28,11 @@ template static std::optional> min_max(Span value if (values.is_empty()) { return std::nullopt; } + const MinMaxResult init{values.first(), values.first()}; return threading::parallel_reduce( values.index_range(), 1024, - MinMaxResult(), + init, [&](IndexRange range, const MinMaxResult &init) { MinMaxResult result = init; for (const int i : range) { @@ -55,10 +56,11 @@ static std::optional> min_max_with_radii(Span values, Span init{values.first(), values.first()}; return threading::parallel_reduce( values.index_range(), 1024, - MinMaxResult(), + init, [&](IndexRange range, const MinMaxResult &init) { MinMaxResult result = init; for (const int i : range) { diff --git a/source/blender/blenlib/tests/BLI_bounds_test.cc b/source/blender/blenlib/tests/BLI_bounds_test.cc index 9c123d4705c..5aa4e710e90 100644 --- a/source/blender/blenlib/tests/BLI_bounds_test.cc +++ b/source/blender/blenlib/tests/BLI_bounds_test.cc @@ -33,6 +33,13 @@ TEST(bounds, MinMaxFloat) EXPECT_EQ(result->max, 3.0f); } +TEST(bounds, MinGreaterThanZero) +{ + Array data = {1.5f, 3.0f, 1.1f, 100.0f}; + auto result = bounds::min_max(data.as_span()); + EXPECT_GT(result->min, 1.0f); +} + TEST(bounds, MinMaxRadii) { Array data = {int2(0, 1), int2(3, -1), int2(0, -2), int2(-1, 1)};