diff options
author | Lioncash <mathew1800@gmail.com> | 2019-05-14 08:09:15 -0400 |
---|---|---|
committer | Lioncash <mathew1800@gmail.com> | 2019-05-14 08:09:17 -0400 |
commit | c5129a3a58956256c72bf3915a2a2ac93a1f58e3 (patch) | |
tree | 7edb0e421653340647833a3b4b50058e2388c0cf | |
parent | c4d549919fbcae062d7bfc6ecb4162fdbcf858f4 (diff) |
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.h | 6 |
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 { CommandDataContainer(CommandData&& data, u64 next_fence) : data{std::move(data)}, fence{next_fence} {} - CommandDataContainer& operator=(const CommandDataContainer& t) { - data = std::move(t.data); - fence = t.fence; - return *this; - } - CommandData data; u64 fence{}; }; |