dnsdist: Make clang-tidy happy by passing the settings by value

diff --git a/pdns/dnsdistdist/dnsdist-cache.cc b/pdns/dnsdistdist/dnsdist-cache.cc
index 79d8d118fe984f350f193b3b6e86c041d18dbaba..12eb6e81ee9838da6e47de7ac02d9639c5ef2609 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-cache.cc
+++ b/pdns/dnsdistdist/dnsdist-cache.cc
@@ -31,8 +31,8 @@
 #include "base64.hh"
 
 // NOLINTNEXTLINE(bugprone-easily-swappable-parameters): too cumbersome to change at this point
-DNSDistPacketCache::DNSDistPacketCache(const CacheSettings& settings) :
-  d_settings(settings)
+DNSDistPacketCache::DNSDistPacketCache(CacheSettings settings) :
+  d_settings(std::move(settings))
 {
   if (d_settings.d_maxEntries == 0) {
     throw std::runtime_error("Trying to create a 0-sized packet-cache");

diff --git a/pdns/dnsdistdist/dnsdist-cache.hh b/pdns/dnsdistdist/dnsdist-cache.hh
index c6e249508acb8d062f2e3d5452c1342c69a4f7d7..41b342aaad83d4e2d36c7ebd27d16cbdd365b3c4 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-cache.hh
+++ b/pdns/dnsdistdist/dnsdist-cache.hh
@@ -52,7 +52,7 @@ public:
     bool d_keepStaleData{false};
   };
 
-  DNSDistPacketCache(const CacheSettings& settings);
+  DNSDistPacketCache(CacheSettings settings);
 
   void insert(uint32_t key, const boost::optional<Netmask>& subnet, uint16_t queryFlags, bool dnssecOK, const DNSName& qname, uint16_t qtype, uint16_t qclass, const PacketBuffer& response, bool receivedOverUDP, uint8_t rcode, boost::optional<uint32_t> tempFailureTTL);
   bool get(DNSQuestion& dnsQuestion, uint16_t queryId, uint32_t* keyOut, boost::optional<Netmask>& subnet, bool dnssecOK, bool receivedOverUDP, uint32_t allowExpired = 0, bool skipAging = false, bool truncatedOK = true, bool recordMiss = true);