clang-tidy: add noexcept for move stuff

Found with performance-noexcept-move-constructor

diff --git a/pdns/credentials.cc b/pdns/credentials.cc
index 5038804b6386a4758727b065c09011b7d0747afb..ec11732f246c04a5998da426becf9ab2b2e4ef3e 100644 (file)
--- a/pdns/credentials.cc
+++ b/pdns/credentials.cc
@@ -69,7 +69,7 @@ SensitiveData::SensitiveData(std::string&& data) :
 #endif
 }
 
-SensitiveData& SensitiveData::operator=(SensitiveData&& rhs)
+SensitiveData& SensitiveData::operator=(SensitiveData&& rhs) noexcept
 {
   d_data = std::move(rhs.d_data);
   rhs.clear();

diff --git a/pdns/credentials.hh b/pdns/credentials.hh
index 6e59c6b40e40effa9b23e8003596a5d5e24e91c4..6762b1a845116d6247bfb9daa835b1083f0c8bda 100644 (file)
--- a/pdns/credentials.hh
+++ b/pdns/credentials.hh
@@ -29,7 +29,7 @@ class SensitiveData
 public:
   SensitiveData(size_t bytes);
   SensitiveData(std::string&& data);
-  SensitiveData& operator=(SensitiveData&&);
+  SensitiveData& operator=(SensitiveData&&) noexcept;
 
   ~SensitiveData();
   void clear();

diff --git a/pdns/dnsname.hh b/pdns/dnsname.hh
index 101d17464c6e68a49e13bfba966179aced1ff0a7..034e3d4f1b446a4bd7f495f578c43a079bcbf3c0 100644 (file)
--- a/pdns/dnsname.hh
+++ b/pdns/dnsname.hh
@@ -89,7 +89,7 @@ public:
     }
     return *this;
   }
-  DNSName& operator=(DNSName&& rhs)
+  DNSName& operator=(DNSName&& rhs) noexcept
   {
     if (this != &rhs) {
       d_storage = std::move(rhs.d_storage);

diff --git a/pdns/dnsrecords.hh b/pdns/dnsrecords.hh
index f61d915266f29ac0317193938681bd69df2fac9a..f674dd4da9696fb6b206a7f4b6fc0e8e3995df79 100644 (file)
--- a/pdns/dnsrecords.hh
+++ b/pdns/dnsrecords.hh
@@ -638,7 +638,8 @@ public:
 
     return *this;
   }
-  NSECBitmap(NSECBitmap&& rhs): d_bitset(std::move(rhs.d_bitset)), d_set(std::move(rhs.d_set))
+  NSECBitmap(NSECBitmap&& rhs) noexcept :
+    d_bitset(std::move(rhs.d_bitset)), d_set(std::move(rhs.d_set))
   {
   }
   bool isSet(uint16_t type) const

diff --git a/pdns/iputils.hh b/pdns/iputils.hh
index ee2f8f04f2d19d69d0ed25291c4aa1b75c84d616..7e6a392105fd0355a00ca01c49d275ffa90ff3f1 100644 (file)
--- a/pdns/iputils.hh
+++ b/pdns/iputils.hh
@@ -1288,7 +1288,8 @@ public:
   }
 
   //<! swaps the contents with another NetmaskTree
-  void swap(NetmaskTree& rhs) {
+  void swap(NetmaskTree& rhs) noexcept
+  {
     std::swap(d_root, rhs.d_root);
     std::swap(d_left, rhs.d_left);
     std::swap(d_size, rhs.d_size);

diff --git a/pdns/lock.hh b/pdns/lock.hh
index 411af5b19bcb1ae44abb2218a95dd9888069270b..611d8ada02a7cecac3956c7488fc602a174dee19 100644 (file)
--- a/pdns/lock.hh
+++ b/pdns/lock.hh
@@ -109,7 +109,8 @@ public:
 
   ReadLock(const ReadLock& rhs) = delete;
   ReadLock& operator=(const ReadLock& rhs) = delete;
-  ReadLock(ReadLock&& rhs) : d_lock(std::move(rhs.d_lock))
+  ReadLock(ReadLock&& rhs) noexcept :
+    d_lock(std::move(rhs.d_lock))
   {
   }
 
@@ -134,7 +135,8 @@ public:
 
   WriteLock(const WriteLock& rhs) = delete;
   WriteLock& operator=(const WriteLock& rhs) = delete;
-  WriteLock(WriteLock&& rhs) : d_lock(std::move(rhs.d_lock))
+  WriteLock(WriteLock&& rhs) noexcept :
+    d_lock(std::move(rhs.d_lock))
   {
   }
 

diff --git a/pdns/sstuff.hh b/pdns/sstuff.hh
index d1f2a5bc217a49c4e3fba67cf005db5c84504043..88f680929934dd1363d6ce5dc577cee4b74836c3 100644 (file)
--- a/pdns/sstuff.hh
+++ b/pdns/sstuff.hh
@@ -58,12 +58,13 @@ public:
     setCloseOnExec(d_socket);
   }
 
-  Socket(Socket&& rhs): d_buffer(std::move(rhs.d_buffer)), d_socket(rhs.d_socket)
+  Socket(Socket&& rhs) noexcept :
+    d_buffer(std::move(rhs.d_buffer)), d_socket(rhs.d_socket)
   {
     rhs.d_socket = -1;
   }
 
-  Socket& operator=(Socket&& rhs)
+  Socket& operator=(Socket&& rhs) noexcept
   {
     if (d_socket != -1) {
       close(d_socket);