commit 8c4623c63a2676e79e7917bb0561f903760f19b9
author Sameer Agarwal <sameeragarwal@google.com> Thu Sep 01 00:05:09 2016 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Sat Sep 03 02:01:10 2016 +0000
tree 1ca49a8051f62aea685a55c65f1b04f6076791d2
parent 2fd39fcecb47eebce727081c9ffb8edf86c33669

Update ExpectArraysClose to use ExpectClose instead of EXPECT_NEAR

The documentation for ExpectArraysClose and its implementation
did not match.

This change makes the polynomial_test not fail on 64bit AMD builds.

Thanks to Phillip Huebner for reporting this.

Change-Id: I503f2d3317a28d5885a34f8bdbccd49d20ae9ba2

internal/ceres/test_util.cc

diff --git a/internal/ceres/test_util.cc b/internal/ceres/test_util.cc
index 5edd4fc..3866eb5 100644
--- a/internal/ceres/test_util.cc
+++ b/internal/ceres/test_util.cc
@@ -117,7 +117,10 @@
   CHECK(q);
 
   for (int i = 0; i < n; ++i) {
-    EXPECT_NEAR(p[i], q[i], tol) << "i=" << i;
+    EXPECT_TRUE(ExpectClose(p[i], q[i], tol))
+        << "p[" << i << "]" << p[i] << " "
+        << "q[" << i << "]" << q[i] << " "
+        << "tol: " << tol;
   }
 }