some d_* locals

diff --git a/pdns/dnspacket.cc b/pdns/dnspacket.cc
index 51f8c7c40bae8d91b4cdbf4324cb0a4e0ea9e4f9..d1707aa990a01c1a3b2ddb35803b914f2cb8e56a 100644 (file)
--- a/pdns/dnspacket.cc
+++ b/pdns/dnspacket.cc
@@ -164,10 +164,10 @@ vector<DNSZoneRecord*> DNSPacket::getAnswerRecords()
 {
   vector<DNSZoneRecord*> arrs;
 
-  for(auto & d_rr : d_rrs)
+  for(auto & rr : d_rrs)
     {
-      if(d_rr.dr.d_place!=DNSResourceRecord::ADDITIONAL)
-        arrs.push_back(&d_rr);
+      if(rr.dr.d_place!=DNSResourceRecord::ADDITIONAL)
+        arrs.push_back(&rr);
     }
   return arrs;
 }
@@ -524,12 +524,12 @@ try
     if(edo.d_extFlags & EDNSOpts::DNSSECOK)
       d_dnssecOk=true;
 
-    for(const auto & d_option : edo.d_options) {
-      if(d_option.first == EDNSOptionCode::NSID) {
+    for(const auto & option : edo.d_options) {
+      if(option.first == EDNSOptionCode::NSID) {
         d_wantsnsid=true;
       }
-      else if(s_doEDNSSubnetProcessing && (d_option.first == EDNSOptionCode::ECS)) { // 'EDNS SUBNET'
-        if(getEDNSSubnetOptsFromString(d_option.second, &d_eso)) {
+      else if(s_doEDNSSubnetProcessing && (option.first == EDNSOptionCode::ECS)) { // 'EDNS SUBNET'
+        if(getEDNSSubnetOptsFromString(option.second, &d_eso)) {
           //cerr<<"Parsed, source: "<<d_eso.source.toString()<<", scope: "<<d_eso.scope.toString()<<", family = "<<d_eso.scope.getNetwork().sin4.sin_family<<endl;
           d_haveednssubnet=true;
         } 

diff --git a/pdns/responsestats.cc b/pdns/responsestats.cc
index 9e5fcdd01051777ab2f9f6199745b507e7ff469d..fefa35c69e5a9f0d8767b8f3ce156fa7f017f413 100644 (file)
--- a/pdns/responsestats.cc
+++ b/pdns/responsestats.cc
@@ -63,8 +63,8 @@ map<uint16_t, uint64_t> ResponseStats::getQTypeResponseCounts()
 map<uint16_t, uint64_t> ResponseStats::getSizeResponseCounts()
 {
   map<uint16_t, uint64_t> ret;
-  for(const auto & d_sizecounter : d_sizecounters) {
-    ret[d_sizecounter.first]=d_sizecounter.second;
+  for(const auto & sizecounter : d_sizecounters) {
+    ret[sizecounter.first]=sizecounter.second;
   }
   return ret;
 }

diff --git a/pdns/signingpipe.cc b/pdns/signingpipe.cc
index 020011ce475ebc46446daa3d34628819cf7b013f..b859bae7ecd975bffac403be98fe04ff85350044 100644 (file)
--- a/pdns/signingpipe.cc
+++ b/pdns/signingpipe.cc
@@ -132,12 +132,12 @@ pair<vector<int>, vector<int> > ChunkedSigningPipe::waitForRW(bool rd, bool wr,
 {
   vector<pollfd> pfds;
 
-  for(int & d_socket : d_sockets) {    
-    if(d_eof.count(d_socket))  
+  for(int & socket : d_sockets) {    
+    if(d_eof.count(socket))  
       continue;
     struct pollfd pfd;
     memset(&pfd, 0, sizeof(pfd));
-    pfd.fd = d_socket;
+    pfd.fd = socket;
     if(rd)
       pfd.events |= POLLIN;
     if(wr)

diff --git a/pdns/stubresolver.cc b/pdns/stubresolver.cc
index 3910c524243dc97b56121b50b0059f74ab453ea0..3951be74b3cbac287b9b4605b7caf0668d73df17 100644 (file)
--- a/pdns/stubresolver.cc
+++ b/pdns/stubresolver.cc
@@ -159,10 +159,10 @@ int stubDoResolve(const DNSName& qname, uint16_t qtype, vector<DNSZoneRecord>& r
     if(mdp.d_header.rcode == RCode::ServFail)
       continue;
 
-    for(const auto & d_answer : mdp.d_answers) {
-      if(d_answer.first.d_place == 1 && d_answer.first.d_type==qtype) {
+    for(const auto & answer : mdp.d_answers) {
+      if(answer.first.d_place == 1 && answer.first.d_type==qtype) {
         DNSZoneRecord zrr;
-        zrr.dr = d_answer.first;
+        zrr.dr = answer.first;
         zrr.auth=true;
         ret.push_back(zrr);
       }

diff --git a/pdns/ws-auth.cc b/pdns/ws-auth.cc
index ef5b9c777acec89da5948e86729fcaa8a0354209..674446ccbfe4b90bdf05fbe14324f3f92a320713 100644 (file)
--- a/pdns/ws-auth.cc
+++ b/pdns/ws-auth.cc
@@ -189,8 +189,8 @@ void AuthWebServer::printargs(ostringstream &ret)
   ret<<R"(<table border=1><tr><td colspan=3 bgcolor="#0000ff"><font color="#ffffff">Arguments</font></td>)"<<endl;
 
   vector<string>entries=arg().list();
-  for(const auto & entrie : entries) {
-    ret<<"<tr><td>"<<entrie<<"</td><td>"<<arg()[entrie]<<"</td><td>"<<arg().getHelp(entrie)<<"</td>"<<endl;
+  for(const auto & entry : entries) {
+    ret<<"<tr><td>"<<entry<<"</td><td>"<<arg()[entry]<<"</td><td>"<<arg().getHelp(entry)<<"</td>"<<endl;
   }
 }