From 3916a1bef7a6c3dd1fc00ec3a2ae75a5af02d204 Mon Sep 17 00:00:00 2001
From: Kefu Chai <kchai@redhat.com>
Date: Sat, 11 Jan 2020 22:41:59 +0800
Subject: [PATCH] common/strtol.cc: refactor strict_si_cast() to silence
 warnings

to silence warnings like

../src/common/strtol.cc:286:7: warning: implicit conversion from 'promoted_t' (aka 'long long') to 'double' changes value from 9223372036854775807 to 9223372036854775808
[-Wimplicit-int-float-conversion]
      static_cast<promoted_t>(std::numeric_limits<T>::max()) / pow (10, m)) {
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~
../src/common/strtol.cc:294:15: note: in instantiation of function template specialization 'strict_si_cast<long>' requested here
template long strict_si_cast<long>(std::string_view str, std::string *err);
              ^

Signed-off-by: Kefu Chai <kchai@redhat.com>
---
 src/common/strtol.cc | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/common/strtol.cc b/src/common/strtol.cc
index 9b437596941f..c39363c88c5f 100644
--- a/src/common/strtol.cc
+++ b/src/common/strtol.cc
@@ -14,6 +14,7 @@
 
 #include "strtol.h"
 
+#include <algorithm>
 #include <climits>
 #include <limits>
 #include <cmath>
@@ -277,17 +278,17 @@ T strict_si_cast(std::string_view str, std::string *err)
     return 0;
   }
   using promoted_t = typename std::common_type<decltype(ll), T>::type;
-  if (static_cast<promoted_t>(ll) <
-      static_cast<promoted_t>(std::numeric_limits<T>::min()) / pow (10, m)) {
-    *err = "strict_sistrtoll: value seems to be too small";
-    return 0;
-  }
-  if (static_cast<promoted_t>(ll) >
-      static_cast<promoted_t>(std::numeric_limits<T>::max()) / pow (10, m)) {
-    *err = "strict_sistrtoll: value seems to be too large";
+  auto v = static_cast<promoted_t>(ll);
+  auto coefficient = static_cast<promoted_t>(powl(10, m));
+  if (v != std::clamp(v,
+		      (static_cast<promoted_t>(std::numeric_limits<T>::min()) /
+		       coefficient),
+		      (static_cast<promoted_t>(std::numeric_limits<T>::max()) /
+		       coefficient))) {
+    *err = "strict_sistrtoll: value out of range";
     return 0;
   }
-  return (ll * pow (10,  m));
+  return v * coefficient;
 }
 
 template int strict_si_cast<int>(std::string_view str, std::string *err);
-- 
2.47.3