From: Rosen Penev <rosenp@gmail.com>
Date: Mon, 9 Dec 2024 00:58:46 +0000 (-0800)
Subject: clang-tidy: use std::min/max
X-Git-Tag: dnsdist-2.0.0-alpha1~152^2~2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3e2324c0c2d16f3056749d81884193d9093352df;p=thirdparty%2Fpdns.git

clang-tidy: use std::min/max

Found with readability-use-std-min-max

Signed-off-by: Rosen Penev <rosenp@gmail.com>
---

diff --git a/pdns/dnspacket.cc b/pdns/dnspacket.cc
index 8588f2c21a..2afde53a35 100644
--- a/pdns/dnspacket.cc
+++ b/pdns/dnspacket.cc
@@ -23,6 +23,7 @@
 #include "config.h"
 #endif
 #include "utility.hh"
+#include <algorithm>
 #include <cstdio>
 #include <cstdlib>
 #include <sys/types.h>
@@ -245,8 +246,7 @@ unsigned int DNSPacket::getMinTTL()
 {
   unsigned int minttl = UINT_MAX;
   for(const DNSZoneRecord& rr :  d_rrs) {
-  if (rr.dr.d_ttl < minttl)
-      minttl = rr.dr.d_ttl;
+    minttl = std::min(minttl, rr.dr.d_ttl);
   }
 
   return minttl;
diff --git a/pdns/dnsparser.cc b/pdns/dnsparser.cc
index 12974ae57c..8a248587d7 100644
--- a/pdns/dnsparser.cc
+++ b/pdns/dnsparser.cc
@@ -1016,9 +1016,7 @@ uint32_t getDNSPacketMinTTL(const char* packet, size_t length, bool* seenAuthSOA
       }
 
       const uint32_t ttl = dpm.get32BitInt();
-      if (result > ttl) {
-        result = ttl;
-      }
+      result = std::min(result, ttl);
 
       dpm.skipRData();
     }
diff --git a/pdns/lua-record.cc b/pdns/lua-record.cc
index 1f2344a881..b06c163061 100644
--- a/pdns/lua-record.cc
+++ b/pdns/lua-record.cc
@@ -1096,18 +1096,14 @@ static void setupLuaRecords(LuaContext& lua) // NOLINT(readability-function-cogn
    * @example ifportup(443, { '1.2.3.4', '5.4.3.2' })"
    */
   lua.writeFunction("ifportup", [](int port, const boost::variant<iplist_t, ipunitlist_t>& ips, const boost::optional<std::unordered_map<string,string>> options) {
-      if (port < 0) {
-        port = 0;
-      }
-      if (port > std::numeric_limits<uint16_t>::max()) {
-        port = std::numeric_limits<uint16_t>::max();
-      }
+    port = std::max(port, 0);
+    port = std::min(port, static_cast<int>(std::numeric_limits<uint16_t>::max()));
 
-      auto checker = [](const ComboAddress& addr, const opts_t& opts) {
-        return g_up.isUp(addr, opts);
-      };
-      return genericIfUp(ips, options, checker, port);
-    });
+    auto checker = [](const ComboAddress& addr, const opts_t& opts) {
+      return g_up.isUp(addr, opts);
+    };
+    return genericIfUp(ips, options, checker, port);
+  });
 
   lua.writeFunction("ifurlextup", [](const vector<pair<int, opts_t> >& ipurls, boost::optional<opts_t> options) {
       vector<ComboAddress> candidates;
diff --git a/pdns/sillyrecords.cc b/pdns/sillyrecords.cc
index cc04a3c29f..a6edc189e2 100644
--- a/pdns/sillyrecords.cc
+++ b/pdns/sillyrecords.cc
@@ -2,6 +2,7 @@
 #include "config.h"
 #endif
 
+#include <algorithm>
 #include <boost/format.hpp>
 
 #include "utility.hh"
@@ -46,8 +47,7 @@ static uint8_t precsize_aton(const char **strptr)
       break;
 
   mantissa = cmval / poweroften[exponent];
-  if (mantissa > 9)
-    mantissa = 9;
+  mantissa = std::min(mantissa, 9);
 
   retval = (mantissa << 4) | exponent;