Process review comments

diff --git a/.clang-tidy.full b/.clang-tidy.full
index e4a95882a62d8b846f7823fc8618fccbe5c2e905..43754f59838a427f3acb02f8741d98892265e2f0 100644 (file)
--- a/.clang-tidy.full
+++ b/.clang-tidy.full
@@ -253,7 +253,7 @@ CheckOptions:
   - key:             modernize-pass-by-value.ValuesOnly
     value:           'false'
   - key:             readability-function-cognitive-complexity.IgnoreMacros
-    value:           'false'
+    value:           'true'
   - key:             modernize-loop-convert.IncludeStyle
     value:           llvm
   - key:             cert-str34-c.DiagnoseSignedUnsignedCharComparisons

diff --git a/pdns/recursordist/rec-main.cc b/pdns/recursordist/rec-main.cc
index cf09ccbb8afa6e3ef2215f26e4354ecc130d57b6..d5f310811314a651fbf6f6aa07000a8d09c2475d 100644 (file)
--- a/pdns/recursordist/rec-main.cc
+++ b/pdns/recursordist/rec-main.cc
@@ -1254,7 +1254,7 @@ void broadcastFunction(const pipefunc_t& func)
       continue;
     }
 
-    ThreadMSG* tmsg = new ThreadMSG(); // NOLINT: manual onwership handling
+    ThreadMSG* tmsg = new ThreadMSG(); // NOLINT: manual ownership handling
     tmsg->func = func;
     tmsg->wantAnswer = true;
     if (write(threadInfo.pipes.writeToThread, &tmsg, sizeof(tmsg)) != sizeof(tmsg)) { // NOLINT: sizeof correct
@@ -1348,7 +1348,7 @@ T broadcastAccFunction(const std::function<T*()>& func)
 
     if (resp) {
       ret += *resp;
-      delete resp; // NOLINT: manual ownershipm handling
+      delete resp; // NOLINT: manual ownership handling
       resp = nullptr;
     }
     // coverity[leaked_storage]
@@ -1915,7 +1915,6 @@ static int serviceMain(Logr::log_t log) // NOLINT(readability-function-cognitive
 
   initDontQuery(log);
 
-  /* this needs to be done before parseACLs(), which call broadcastFunction() */
   RecThreadInfo::setWeDistributeQueries(::arg().mustDo("pdns-distributes-queries"));
   if (RecThreadInfo::weDistributeQueries()) {
     SLOG(g_log << Logger::Warning << "PowerDNS Recursor itself will distribute queries over threads" << endl,