clang-tidy tweaks

diff --git a/pdns/recursordist/recursor_cache.cc b/pdns/recursordist/recursor_cache.cc
index 0c240e71ed9acf4684aae4578a8ab5b6f09d2409..9bfdb010fd1037d2ed496b678988550c600ba87b 100644 (file)
--- a/pdns/recursordist/recursor_cache.cc
+++ b/pdns/recursordist/recursor_cache.cc
@@ -578,7 +578,7 @@ bool MemRecursorCache::replace(CacheEntry&& entry)
 
   lockedShard->d_cachecachevalid = false;
   entry.d_submitted = false;
-  if (lockedShard->d_map.insert(entry).second) {
+  if (lockedShard->d_map.emplace(std::move(entry)).second) {
     shard.incEntriesCount();
     return true;
   }

diff --git a/pdns/recursordist/recursor_cache.hh b/pdns/recursordist/recursor_cache.hh
index 50c637afdf89f7cf0e48a0026b713a2323603620..596eeb85ad70def72720c7e6480bb349bfd076d4 100644 (file)
--- a/pdns/recursordist/recursor_cache.hh
+++ b/pdns/recursordist/recursor_cache.hh
@@ -21,7 +21,6 @@
  */
 #pragma once
 #include <string>
-#include <set>
 #include "dns.hh"
 #include "qtype.hh"
 #include "misc.hh"
@@ -63,7 +62,7 @@ public:
   [[nodiscard]] pair<uint64_t, uint64_t> stats();
   [[nodiscard]] size_t ecsIndexSize();
 
-  size_t getRecords(size_t howmany, size_t maxsize, std::string& ret);
+  size_t getRecords(size_t perShard, size_t maxSize, std::string& ret);
   size_t putRecords(const std::string& pbuf);
 
   using OptTag = boost::optional<std::string>;