Process comments from review

diff --git a/pdns/pdns_recursor.cc b/pdns/pdns_recursor.cc
index be3792a6209768ea604c1f1562af87ecae6f1409..3d68a175239a33a57e8f5b7dd3e37bf11858f793 100644 (file)
--- a/pdns/pdns_recursor.cc
+++ b/pdns/pdns_recursor.cc
@@ -1843,10 +1843,10 @@ bool checkForCacheHit(bool qnameParsed, unsigned int tag, const string& data,
 
     // This is only to get the proxyMapping suffixMatch stats right i the case of a PC hit
     if (t_proxyMapping && source != mappedSource) {
-      if (auto it = t_proxyMapping->lookup(source)) {
-        if (it->second.suffixMatchNode) {
-          if (it->second.suffixMatchNode->check(qname)) {
-            ++it->second.stats.suffixMatches;
+      if (const auto* found = t_proxyMapping->lookup(source)) {
+        if (found->second.suffixMatchNode) {
+          if (found->second.suffixMatchNode->check(qname)) {
+            ++found->second.stats.suffixMatches;
           }
         }
       }

diff --git a/pdns/rec_channel_rec.cc b/pdns/rec_channel_rec.cc
index 3c96649c2c155c5100d8083ca21cf8c47f10aa82..a403aeef88fd59f8caa3f0f03db63ed6ec45473f 100644 (file)
--- a/pdns/rec_channel_rec.cc
+++ b/pdns/rec_channel_rec.cc
@@ -942,8 +942,9 @@ static ProxyMappingStats_t* pleaseGetProxyMappingStats()
 static string doGetProxyMappingStats()
 {
   ostringstream ret;
-  auto m = broadcastAccFunction<ProxyMappingStats_t>(pleaseGetProxyMappingStats);
-  for (const auto& [key, entry] : m) {
+  ret << "subnet\t\t\tmatches\tsuffixmatches" << endl;
+  auto proxyMappingStats = broadcastAccFunction<ProxyMappingStats_t>(pleaseGetProxyMappingStats);
+  for (const auto& [key, entry] : proxyMappingStats) {
     ret << key.toString() << '\t' << entry.netmaskMatches << '\t' << entry.suffixMatches << endl;
   }
   return ret.str();
@@ -1164,9 +1165,9 @@ static StatsMap toProxyMappingStatsMap(const string& name)
   const string pbasename = getPrometheusName(name);
   StatsMap entries;
 
-  auto m = broadcastAccFunction<ProxyMappingStats_t>(pleaseGetProxyMappingStats);
+  auto proxyMappingStats = broadcastAccFunction<ProxyMappingStats_t>(pleaseGetProxyMappingStats);
   size_t count = 0;
-  for (const auto& [key, entry] : m) {
+  for (const auto& [key, entry] : proxyMappingStats) {
     auto keyname = pbasename + "{netmask=\"" + key.toString() + "\",count=\"";
     auto sname1 = name + "-n-" + std::to_string(count);
     auto pname1 = keyname + "netmaskmatches\"}";
@@ -1920,7 +1921,7 @@ static void* pleaseSupplantProxyMapping(const ProxyMapping& pm)
       auto& newentry = newmapping->insert(nm);
       newentry.second = entry;
       if (t_proxyMapping) {
-        if (auto existing = t_proxyMapping->lookup(nm); existing != nullptr) {
+        if (const auto* existing = t_proxyMapping->lookup(nm); existing != nullptr) {
           newentry.second.stats = existing->second.stats;
         }
       }