aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--changes/bug228015
-rw-r--r--src/common/util.c12
2 files changed, 17 insertions, 0 deletions
diff --git a/changes/bug22801 b/changes/bug22801
new file mode 100644
index 0000000000..7edc79bc84
--- /dev/null
+++ b/changes/bug22801
@@ -0,0 +1,5 @@
+ o Minor bugfixes (compilation):
+ - When building with certain versions the mingw C header files, avoid
+ float-conversion warnings when calling the C functions isfinite(),
+ isnan(), and signbit(). Fixes bug 22801; bugfix on 0.2.8.1-alpha.
+
diff --git a/src/common/util.c b/src/common/util.c
index f3effe0957..b2b97613a7 100644
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -5537,6 +5537,15 @@ clamp_double_to_int64(double number)
{
int exp;
+#if defined(__MINGW32__) || defined(__MINGW64__)
+/*
+ Mingw's math.h uses gcc's __builtin_choose_expr() facility to declare
+ isnan, isfinite, and signbit. But as implemented in at least some
+ versions of gcc, __builtin_choose_expr() can generate type warnings
+ even from branches that are not taken. So, suppress those warnings.
+*/
+DISABLE_GCC_WARNING(float-conversion)
+#endif
/* NaN is a special case that can't be used with the logic below. */
if (isnan(number)) {
return 0;
@@ -5562,5 +5571,8 @@ clamp_double_to_int64(double number)
/* Handle infinities and finite numbers with magnitude >= 2^63. */
return signbit(number) ? INT64_MIN : INT64_MAX;
+#if defined(__MINGW32__) || defined(__MINGW64__)
+ENABLE_GCC_WARNING(float-conversion)
+#endif
}