diff options
| author | 2018-04-29 18:29:03 -0400 | |
|---|---|---|
| committer | 2018-04-29 18:34:09 -0400 | |
| commit | e8bbafb746ce7e178be757471305539c05bb7f23 (patch) | |
| tree | 5396d4f28614f5b346a21776935597ec466f117f /src/common/file_util.cpp | |
| parent | file_util: Add static assertions to ReadBytes() and WriteBytes() (diff) | |
| download | yuzu-e8bbafb746ce7e178be757471305539c05bb7f23.tar.gz yuzu-e8bbafb746ce7e178be757471305539c05bb7f23.tar.xz yuzu-e8bbafb746ce7e178be757471305539c05bb7f23.zip | |
file_util: Make move constructor/assignment operator and related functions noexcept
Without this, it's possible to get compilation failures in the (rare) scenario where
a container is used to store a bunch of live IOFile instances, as they may be using
std::move_if_noexcept under the hood. Given these definitely don't throw exceptions
this is also not incorrect to add either.
Diffstat (limited to '')
| -rw-r--r-- | src/common/file_util.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/common/file_util.cpp b/src/common/file_util.cpp index cd852bfd8..2d0b81c6e 100644 --- a/src/common/file_util.cpp +++ b/src/common/file_util.cpp | |||
| @@ -809,16 +809,16 @@ IOFile::~IOFile() { | |||
| 809 | Close(); | 809 | Close(); |
| 810 | } | 810 | } |
| 811 | 811 | ||
| 812 | IOFile::IOFile(IOFile&& other) { | 812 | IOFile::IOFile(IOFile&& other) noexcept { |
| 813 | Swap(other); | 813 | Swap(other); |
| 814 | } | 814 | } |
| 815 | 815 | ||
| 816 | IOFile& IOFile::operator=(IOFile&& other) { | 816 | IOFile& IOFile::operator=(IOFile&& other) noexcept { |
| 817 | Swap(other); | 817 | Swap(other); |
| 818 | return *this; | 818 | return *this; |
| 819 | } | 819 | } |
| 820 | 820 | ||
| 821 | void IOFile::Swap(IOFile& other) { | 821 | void IOFile::Swap(IOFile& other) noexcept { |
| 822 | std::swap(m_file, other.m_file); | 822 | std::swap(m_file, other.m_file); |
| 823 | std::swap(m_good, other.m_good); | 823 | std::swap(m_good, other.m_good); |
| 824 | } | 824 | } |