Skip to content

Commit

Permalink
nolint the bug for now, fix in another PR since it might change a bit
Browse files Browse the repository at this point in the history
  • Loading branch information
TrentHouliston committed Dec 26, 2024
1 parent e3cbbee commit 2d0cc4d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/util/FunctionFusion.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,11 @@ namespace util {
NextStep::call(std::forward<Args>(args)...))) {

// Call each on a separate line to preserve order of execution
auto current = tuplify(call_one<CurrentFunction>(CurrentRange(), std::forward<Args>(args)...));
// TODO(thouliston) this is a legitimate bug, fix it in a future PR and add a test to ensure it doesn't
// regress
// NOLINTNEXTLINE(bugprone-use-after-move)
auto current = tuplify(call_one<CurrentFunction>(CurrentRange(), std::forward<Args>(args)...));
// NOLINTNEXTLINE(bugprone-use-after-move)
auto remainder = NextStep::call(std::forward<Args>(args)...);

return std::tuple_cat(std::move(current), std::move(remainder));
Expand Down

0 comments on commit 2d0cc4d

Please sign in to comment.