diff options
| author | Lioncash <mathew1800@gmail.com> | 2018-04-29 18:29:03 -0400 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2018-04-29 18:34:09 -0400 |
| commit | e8bbafb746ce7e178be757471305539c05bb7f23 (patch) | |
| tree | 5396d4f28614f5b346a21776935597ec466f117f /src/common/file_util.h | |
| parent | 40d2dcabd7e5b978c0e1e5c76000de01e2d0c270 (diff) | |
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 'src/common/file_util.h')
| -rw-r--r-- | src/common/file_util.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/common/file_util.h b/src/common/file_util.h index 32ff4d8ca2..fc6b3ea466 100644 --- a/src/common/file_util.h +++ b/src/common/file_util.h @@ -160,10 +160,10 @@ public: ~IOFile(); - IOFile(IOFile&& other); - IOFile& operator=(IOFile&& other); + IOFile(IOFile&& other) noexcept; + IOFile& operator=(IOFile&& other) noexcept; - void Swap(IOFile& other); + void Swap(IOFile& other) noexcept; bool Open(const std::string& filename, const char openmode[]); bool Close(); |