use if() instead of ?: and avoid possible optimization problems

diff --git a/Zend/zend_operators.c b/Zend/zend_operators.c
index 297b3a629a80283c311914ac79f977bf099e338c..7c346831ea0f270b24f0e79ac555fdd5533f8978 100644 (file)
--- a/Zend/zend_operators.c
+++ b/Zend/zend_operators.c
@@ -221,8 +221,13 @@ ZEND_API void convert_scalar_to_number(zval *op TSRMLS_DC)
                }                                                                                                                                       \
        }
 
+#define DVAL_TO_LVAL(d, l) \
+       if ((d) > LONG_MAX) { \
+               (l) = (unsigned long) (d); \
+       } else { \
+               (l) = (long) (d); \
+       }
 
-#define DVAL_TO_LVAL(d, l) (l) = (((d) > LONG_MAX) ? (unsigned long) (d) : (long) (d))
 
 #define zendi_convert_to_long(op, holder, result)                                      \
        if (op == result) {                                                                                             \