Skip to content

Commit

Permalink
#128: render: remove unnecessary std::move to allow copy elision
Browse files Browse the repository at this point in the history
  • Loading branch information
pierrepebay committed Dec 17, 2024
1 parent f3a6fb3 commit 05dfe3d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/vt-tv/render/render.cc
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,7 @@ vtkNew<U> Render::createRankArrayUserDefined(
//fmt::print("phase={}, key={}, rank_id={}\n", phase, key, rank_id);
array->SetTuple1(rank_id, std::get<T>(value));
}
return std::move(array);
return array;
}

template <typename T, typename U>
Expand All @@ -329,7 +329,7 @@ vtkNew<U> Render::createRankArrayComputed(
for (uint64_t rank_id = 0; rank_id < n_ranks_; rank_id++) {
array->SetTuple1(rank_id, info_.getRankQOIAtPhase<T>(rank_id, phase, key));
}
return std::move(array);
return array;
}

vtkNew<vtkPolyData> Render::createRankMesh_(PhaseType phase) {
Expand Down

0 comments on commit 05dfe3d

Please sign in to comment.