summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Lioncash2019-05-14 08:09:15 -0400
committerGravatar Lioncash2019-05-14 08:09:17 -0400
commitc5129a3a58956256c72bf3915a2a2ac93a1f58e3 (patch)
tree7edb0e421653340647833a3b4b50058e2388c0cf
parentMerge pull request #2462 from lioncash/video-mm (diff)
downloadyuzu-c5129a3a58956256c72bf3915a2a2ac93a1f58e3.tar.gz
yuzu-c5129a3a58956256c72bf3915a2a2ac93a1f58e3.tar.xz
yuzu-c5129a3a58956256c72bf3915a2a2ac93a1f58e3.zip
video_core/gpu_thread: Remove redundant copy constructor for CommandDataContainer
std::move within a copy constructor (on a data member that isn't mutable) will always result in a copy. Because of that, the behavior of this copy constructor is identical to the one that would be generated automatically by the compiler, so we can remove it.
-rw-r--r--src/video_core/gpu_thread.h6
1 files changed, 0 insertions, 6 deletions
diff --git a/src/video_core/gpu_thread.h b/src/video_core/gpu_thread.h
index cc14527c7..64a3335ba 100644
--- a/src/video_core/gpu_thread.h
+++ b/src/video_core/gpu_thread.h
@@ -81,12 +81,6 @@ struct CommandDataContainer {
81 CommandDataContainer(CommandData&& data, u64 next_fence) 81 CommandDataContainer(CommandData&& data, u64 next_fence)
82 : data{std::move(data)}, fence{next_fence} {} 82 : data{std::move(data)}, fence{next_fence} {}
83 83
84 CommandDataContainer& operator=(const CommandDataContainer& t) {
85 data = std::move(t.data);
86 fence = t.fence;
87 return *this;
88 }
89
90 CommandData data; 84 CommandData data;
91 u64 fence{}; 85 u64 fence{};
92}; 86};