]> git.ipfire.org Git - thirdparty/ccache.git/commitdiff
Add noexcept to move constructors and assignment operators
authorJoel Rosdahl <joel@rosdahl.net>
Tue, 8 Sep 2020 17:56:54 +0000 (19:56 +0200)
committerJoel Rosdahl <joel@rosdahl.net>
Tue, 8 Sep 2020 18:31:54 +0000 (20:31 +0200)
src/Fd.hpp
src/File.hpp
src/TemporaryFile.hpp

index 9cbc1351dc642f6f626f5687b1a8a8bc39b750dc..e9d139c138a8a9086fcb51d89a18d941a1089438 100644 (file)
@@ -27,7 +27,7 @@ class Fd : NonCopyable
 public:
   Fd() = default;
   explicit Fd(int fd);
-  Fd(Fd&& other_fd);
+  Fd(Fd&& other_fd) noexcept;
   ~Fd();
 
   explicit operator bool() const;
@@ -35,7 +35,7 @@ public:
   int get() const;
   int operator*() const;
 
-  Fd& operator=(Fd&& other_fd);
+  Fd& operator=(Fd&& other_fd) noexcept;
 
   // Close wrapped fd before the lifetime of Fd has ended.
   bool close();
@@ -51,7 +51,7 @@ inline Fd::Fd(int fd) : m_fd(fd)
 {
 }
 
-inline Fd::Fd(Fd&& other_fd) : m_fd(other_fd.release())
+inline Fd::Fd(Fd&& other_fd) noexcept : m_fd(other_fd.release())
 {
 }
 
@@ -81,7 +81,7 @@ Fd::operator*() const
 }
 
 inline Fd&
-Fd::operator=(Fd&& other_fd)
+Fd::operator=(Fd&& other_fd) noexcept
 {
   close();
   m_fd = other_fd.release();
index d021dd08fcc8e32f99288bb0d9c71aea4898fb08..6f0dd21cdd7ffae60a86ef2223b6f933de661ac8 100644 (file)
@@ -29,10 +29,10 @@ class File : public NonCopyable
 public:
   File() = default;
   File(const std::string& path, const char* mode);
-  File(File&& other);
+  File(File&& other) noexcept;
   ~File();
 
-  File& operator=(File&& other);
+  File& operator=(File&& other) noexcept;
 
   void open(const std::string& path, const char* mode);
   void close();
@@ -50,7 +50,7 @@ inline File::File(const std::string& path, const char* mode)
   open(path, mode);
 }
 
-inline File::File(File&& other) : m_file(other.m_file)
+inline File::File(File&& other) noexcept : m_file(other.m_file)
 {
   other.m_file = nullptr;
 }
@@ -61,7 +61,7 @@ inline File::~File()
 }
 
 inline File&
-File::operator=(File&& other)
+File::operator=(File&& other) noexcept
 {
   m_file = other.m_file;
   other.m_file = nullptr;
index d02eafab501798957738a0c886ce32830398fa99..ca40b4821134060fe5ffce3d780dafbd4868d5e1 100644 (file)
@@ -32,9 +32,9 @@ public:
   //  the directory will be created if possible.`
   TemporaryFile(nonstd::string_view path_prefix);
 
-  TemporaryFile(TemporaryFile&& other) = default;
+  TemporaryFile(TemporaryFile&& other) noexcept = default;
 
-  TemporaryFile& operator=(TemporaryFile&& other) = default;
+  TemporaryFile& operator=(TemporaryFile&& other) noexcept = default;
 
   // The resulting open file descriptor in read/write mode. Unset on error.
   Fd fd;