math_clamp_safe() should always print a warning upon invalid bounds

1 files changed, 1 insertions, 3 deletions

diff --git a/src/util/math.h b/src/util/math.h
index f2c839069c..24078775e6 100644
--- a/src/util/math.h
+++ b/src/util/math.h
@@ -9,8 +9,6 @@
 
 #include <QtDebug>
 
-#include "util/cmdlineargs.h"
-
 // If we don't do this then we get the C90 fabs from the global namespace which
 // is only defined for double.
 using std::fabs;
@@ -28,7 +26,7 @@ template <typename T>
 inline T math_clamp_safe(T value, T min, T max) {
     // XXX: If max < min, behavior is undefined, and has been causing problems.
     // if debugging is on, assert when this happens.
-    if (CmdlineArgs::Instance().getDeveloper() && max < min) {
+    if (max < min) {
         qWarning() << "PROGRAMMING ERROR: math_clamp called with max < min! "
                    << max << " " << min;
     }