commit 61e0977e10f49d43e76b1de0f9433379fa985d96
author Christoph Hertzberg <chtz@informatik.uni-bremen.de> Fri Aug 14 17:32:34 2015 +0200
committer Christoph Hertzberg <chtz@informatik.uni-bremen.de> Fri Aug 14 17:32:34 2015 +0200
tree d59991d1338db8bd3e30e5f1a3cc30acd9bbf736
parent 712e2fed1705746847d1cbdce01094eb42a45139

Protect all calls to isnan, isinf and isfinite with parentheses.

test/array.cpp

diff --git a/test/array.cpp b/test/array.cpp
index 3d00d1d..9f61c4b 100644
--- a/test/array.cpp
+++ b/test/array.cpp
@@ -221,9 +221,9 @@
   VERIFY_IS_APPROX(m1.round(), round(m1));
   VERIFY_IS_APPROX(m1.floor(), floor(m1));
   VERIFY_IS_APPROX(m1.ceil(), ceil(m1));
-  VERIFY((m1.isNaN() == Eigen::isnan(m1)).all());
-  VERIFY((m1.isInf() == Eigen::isinf(m1)).all());
-  VERIFY((m1.isFinite() == Eigen::isfinite(m1)).all());
+  VERIFY((m1.isNaN() == (Eigen::isnan)(m1)).all());
+  VERIFY((m1.isInf() == (Eigen::isinf)(m1)).all());
+  VERIFY((m1.isFinite() == (Eigen::isfinite)(m1)).all());
   VERIFY_IS_APPROX(m1.inverse(), inverse(m1));
   VERIFY_IS_APPROX(m1.abs(), abs(m1));
   VERIFY_IS_APPROX(m1.abs2(), abs2(m1));
@@ -249,9 +249,9 @@
   VERIFY_IS_APPROX(tanh(m1), (0.5*(exp(m1)-exp(-m1)))/(0.5*(exp(m1)+exp(-m1))));
   VERIFY_IS_APPROX(arg(m1), ((ArrayType)(m1<0))*std::acos(-1.0));
   VERIFY((round(m1) <= ceil(m1) && round(m1) >= floor(m1)).all());
-  VERIFY(Eigen::isnan((m1*0.0)/0.0).all());
-  VERIFY(Eigen::isinf(m4/0.0).all());
-  VERIFY((Eigen::isfinite(m1) && (!Eigen::isfinite(m1*0.0/0.0)) && (!Eigen::isfinite(m4/0.0))).all());
+  VERIFY((Eigen::isnan)((m1*0.0)/0.0).all());
+  VERIFY((Eigen::isinf)(m4/0.0).all());
+  VERIFY(((Eigen::isfinite)(m1) && (!(Eigen::isfinite)(m1*0.0/0.0)) && (!(Eigen::isfinite)(m4/0.0))).all());
   VERIFY_IS_APPROX(inverse(inverse(m1)),m1);
   VERIFY((abs(m1) == m1 || abs(m1) == -m1).all());
   VERIFY_IS_APPROX(m3, sqrt(abs2(m1)));
@@ -336,9 +336,9 @@
   VERIFY_IS_APPROX(m1.cosh(), cosh(m1));
   VERIFY_IS_APPROX(m1.tanh(), tanh(m1));
   VERIFY_IS_APPROX(m1.arg(), arg(m1));
-  VERIFY((m1.isNaN() == Eigen::isnan(m1)).all());
-  VERIFY((m1.isInf() == Eigen::isinf(m1)).all());
-  VERIFY((m1.isFinite() == Eigen::isfinite(m1)).all());
+  VERIFY((m1.isNaN() == (Eigen::isnan)(m1)).all());
+  VERIFY((m1.isInf() == (Eigen::isinf)(m1)).all());
+  VERIFY((m1.isFinite() == (Eigen::isfinite)(m1)).all());
   VERIFY_IS_APPROX(m1.inverse(), inverse(m1));
   VERIFY_IS_APPROX(m1.log(), log(m1));
   VERIFY_IS_APPROX(m1.log10(), log10(m1));
@@ -364,20 +364,20 @@
   VERIFY_IS_APPROX(arg(m1), m3);
 
   std::complex<RealScalar> zero(0.0,0.0);
-  VERIFY(Eigen::isnan(m1*zero/zero).all());
+  VERIFY((Eigen::isnan)(m1*zero/zero).all());
 #if EIGEN_COMP_CLANG
   // clang's complex division is notoriously broken
-  if(numext::isinf(m4(0,0)/RealScalar(0))) {
+  if((numext::isinf)(m4(0,0)/RealScalar(0))) {
 #endif
-  VERIFY(Eigen::isinf(m4/zero).all());
+  VERIFY((Eigen::isinf)(m4/zero).all());
 #if EIGEN_COMP_CLANG
   }
   else
   {
-    VERIFY(Eigen::isinf(m4.real()/zero.real()).all());
+    VERIFY((Eigen::isinf)(m4.real()/zero.real()).all());
   }
 #endif
-  VERIFY((Eigen::isfinite(m1) && (!Eigen::isfinite(m1*zero/zero)) && (!Eigen::isfinite(m1/zero))).all());
+  VERIFY(((Eigen::isfinite)(m1) && (!(Eigen::isfinite)(m1*zero/zero)) && (!(Eigen::isfinite)(m1/zero))).all());
 
   VERIFY_IS_APPROX(inverse(inverse(m1)),m1);
   VERIFY_IS_APPROX(conj(m1.conjugate()), m1);