diff options
| author | 2019-05-14 08:09:15 -0400 | |
|---|---|---|
| committer | 2019-05-14 08:09:17 -0400 | |
| commit | c5129a3a58956256c72bf3915a2a2ac93a1f58e3 (patch) | |
| tree | 7edb0e421653340647833a3b4b50058e2388c0cf | |
| parent | Merge pull request #2462 from lioncash/video-mm (diff) | |
| download | yuzu-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.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 { | |||
| 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 | }; |