Skip to content

Commit 4bc810c

Browse files
Merge pull request #281 from szadam/CID436576
Fix: Use move instead of copy for args
2 parents 74fb240 + f204690 commit 4bc810c

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

test/common/base.hpp

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,14 @@ template <typename T> T generateArg() { return T{}; }
3838
// with all arguments created by calling generateArg()
3939
template <typename Ret, typename... Args>
4040
std::function<Ret(void)> withGeneratedArgs(Ret (*f)(Args...)) {
41-
std::tuple<Args...> tuple = {};
42-
auto args = std::apply(
43-
[](auto... x) {
44-
return std::make_tuple(generateArg<decltype(x)>()...);
45-
},
46-
tuple);
47-
return [=]() { return std::apply(f, args); };
41+
return [f]() {
42+
auto args = std::apply(
43+
[](auto... x) {
44+
return std::make_tuple(generateArg<decltype(x)>()...);
45+
},
46+
std::tuple<Args...>{});
47+
return std::apply(f, std::move(args));
48+
};
4849
}
4950

5051
} // namespace umf_test

0 commit comments

Comments
 (0)