Warn, don't crash, if math_clamp is used badly

1 files changed, 5 insertions, 1 deletions

diff --git a/src/util/math.h b/src/util/math.h
index b425e2cd9b..dee5a90aff 100644
--- a/src/util/math.h
+++ b/src/util/math.h
@@ -6,6 +6,7 @@
 #define _USE_MATH_DEFINES
 #include <cmath>
 #include <algorithm>
+#include <QDebug>
 
 // If we don't do this then we get the C90 fabs from the global namespace which
 // is only defined for double.
@@ -19,7 +20,10 @@ template <typename T>
 inline T math_clamp(const T& value, const T& min, const T& max) {
     // XXX: If max < min, behavior is undefined, and has been causing problems.
     // if debugging is on, assert when this happens.
-    Q_ASSERT(max >= min);
+    if (max < min) {
+        qWarning() << "PROGRAMMING ERROR: math_clamp called with max < min! "
+                   << max << " " << min;
+    }
     if (value > max) {
         return max;
     }