From 440d949761638248b537a4b9035c27bd528ffbc3 Mon Sep 17 00:00:00 2001
From: Lukas Zanner <git@lukaszanner.de>
Date: Wed, 6 Mar 2024 10:20:25 +0100
Subject: [PATCH] Disable MSVC warning 4127 locally instead of globally

---
 include/soci/soci-platform.h |  4 ----
 include/soci/type-holder.h   | 10 +++++++++-
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/include/soci/soci-platform.h b/include/soci/soci-platform.h
index 125f772fe..cd0a21596 100644
--- a/include/soci/soci-platform.h
+++ b/include/soci/soci-platform.h
@@ -37,10 +37,6 @@
 //base class must have dll interface
 #pragma warning(disable:4251 4275)
 
-// As long as we don't require C++17, we must disable the warning
-// "conditional expression is constant"
-#pragma warning(disable:4127)
-
 // Define if you have the vsnprintf variants.
 #if _MSC_VER < 1500
 # define vsnprintf _vsnprintf
diff --git a/include/soci/type-holder.h b/include/soci/type-holder.h
index 7b366c302..121e11227 100644
--- a/include/soci/type-holder.h
+++ b/include/soci/type-holder.h
@@ -10,7 +10,6 @@
 
 #include "soci/error.h"
 #include "soci/soci-backend.h"
-#include "soci/soci-platform.h"
 #include "soci/soci-types.h"
 
 #include <cstdint>
@@ -89,11 +88,20 @@ struct soci_cast<
         uintmax_t t_max = static_cast<uintmax_t>((std::numeric_limits<T>::max)());
         uintmax_t u_max = static_cast<uintmax_t>((std::numeric_limits<U>::max)());
 
+#ifdef _MSC_VER
+// As long as we don't require C++17, we must disable the warning
+// "conditional expression is constant" as it can give false positives here.
+#pragma warning(push)
+#pragma warning(disable:4127)
+#endif
         if ((t_min > u_min && val < static_cast<U>(t_min)) ||
             (t_max < u_max && val > static_cast<U>(t_max)))
         {
             throw std::bad_cast();
         }
+#ifdef _MSC_VER
+#pragma warning(pop)
+#endif
 
         return static_cast<T>(val);
     }