From: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Date: Wed, 10 Jan 2024 13:54:08 +0000 (+0100)
Subject: Reformat and tidy annotations
X-Git-Tag: dnsdist-1.9.0-rc1~20^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F13702%2Fhead;p=thirdparty%2Fpdns.git

Reformat and tidy annotations
---

diff --git a/pdns/logger.hh b/pdns/logger.hh
index 37216584dc..9a84661442 100644
--- a/pdns/logger.hh
+++ b/pdns/logger.hh
@@ -185,24 +185,30 @@ using OptLog = std::optional<LogVariant>;
 
 void addTraceTS(const timeval& start, ostringstream& str);
 
-#define VLOG(log, x)                                                    \
-  if (log) {                                                            \
-    if (std::holds_alternative<std::reference_wrapper<Logger>>((log)->v)) { \
+// NOLINTNEXTLINE(cppcoreguidelines-macro-usage)
+#define VLOG(log, x)                                                                                     \
+  if (log) {                                                                                             \
+    if (std::holds_alternative<std::reference_wrapper<Logger>>((log)->v)) {                              \
+      /* NOLINTNEXTLINE(bugprone-macro-parentheses) */                                                   \
       std::get<std::reference_wrapper<Logger>>((log)->v).get() << Logger::Warning << (log)->prefix << x; \
-    }                                                                   \
-    else if (std::holds_alternative<std::reference_wrapper<ostringstream>>((log)->v)) { \
-      addTraceTS((log)->start, std::get<std::reference_wrapper<ostringstream>>((log)->v).get()); \
-      std::get<std::reference_wrapper<ostringstream>>((log)->v).get() << (log)->prefix << x; \
-    }                                                                   \
+    }                                                                                                    \
+    else if (std::holds_alternative<std::reference_wrapper<ostringstream>>((log)->v)) {                  \
+      addTraceTS((log)->start, std::get<std::reference_wrapper<ostringstream>>((log)->v).get());         \
+      /* NOLINTNEXTLINE(bugprone-macro-parentheses) */                                                   \
+      std::get<std::reference_wrapper<ostringstream>>((log)->v).get() << (log)->prefix << x;             \
+    }                                                                                                    \
   }
 
-#define VLOG_NO_PREFIX(log, x)                                          \
-  if (log) {                                                            \
-    if (std::holds_alternative<std::reference_wrapper<Logger>>((log)->v)) { \
-      std::get<std::reference_wrapper<Logger>>((log)->v).get() << Logger::Warning << x; \
-    }                                                                   \
-    else if (std::holds_alternative<std::reference_wrapper<ostringstream>>((log)->v)) { \
+// NOLINTNEXTLINE(cppcoreguidelines-macro-usage)
+#define VLOG_NO_PREFIX(log, x)                                                                   \
+  if (log) {                                                                                     \
+    if (std::holds_alternative<std::reference_wrapper<Logger>>((log)->v)) {                      \
+      /* NOLINTNEXTLINE(bugprone-macro-parentheses) */                                           \
+      std::get<std::reference_wrapper<Logger>>((log)->v).get() << Logger::Warning << x;          \
+    }                                                                                            \
+    else if (std::holds_alternative<std::reference_wrapper<ostringstream>>((log)->v)) {          \
       addTraceTS((log)->start, std::get<std::reference_wrapper<ostringstream>>((log)->v).get()); \
-      std::get<std::reference_wrapper<ostringstream>>((log)->v).get() << x; \
-    }                                                                   \
+      /* NOLINTNEXTLINE(bugprone-macro-parentheses) */                                           \
+      std::get<std::reference_wrapper<ostringstream>>((log)->v).get() << x;                      \
+    }                                                                                            \
   }