From c0b5ffe54ceb7ecc16f8c39c29a32cfaa4f4f238 Mon Sep 17 00:00:00 2001
From: Daniel Lemire <lemire@gmail.com>
Date: Mon, 2 Nov 2020 20:22:08 -0500
Subject: [PATCH] Making errors verbose.

---
 tests/basictest.cpp             | 9 +++++----
 tests/exhaustive32_64.cpp       | 1 +
 tests/exhaustive32_midpoint.cpp | 1 +
 tests/long_exhaustive32.cpp     | 1 +
 tests/long_random64.cpp         | 1 +
 tests/random64.cpp              | 1 +
 6 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/tests/basictest.cpp b/tests/basictest.cpp
index 39aefc4..cd46a59 100644
--- a/tests/basictest.cpp
+++ b/tests/basictest.cpp
@@ -46,19 +46,19 @@ bool basic_test_32bit(std::string vals, float val) {
     std::cerr << " I could not parse " << vals << std::endl;
     return false;
   }
-  std::cout << copysign(1,result_value) << std::endl;
-    std::cout << copysign(1,val) << std::endl;
-
   if(copysign(1,result_value) != copysign(1,val)) {
+    std::cerr << vals << std::endl;
     std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << val
               << std::endl;
     return false;
   } else if (std::isnan(val)) {
     if (!std::isnan(result_value)) {
+      std::cerr << vals << std::endl;
       std::cerr << "not nan" << result_value << std::endl;
       return false;
     }
   } else if (result_value != val) {
+    std::cerr << vals << std::endl;
     std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << val
               << std::endl;
     std::cerr << std::dec;
@@ -96,10 +96,12 @@ bool basic_test_64bit(std::string vals, double val) {
     return false;
   } else if (std::isnan(val)) {
     if (!std::isnan(result_value)) {
+      std::cerr << vals << std::endl;
       std::cerr << "not nan" << result_value << std::endl;
       return false;
     }
   } else if (result_value != val) {
+    std::cerr << vals << std::endl;
     std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << val
               << std::endl;
     std::cerr << std::dec;
@@ -107,7 +109,6 @@ bool basic_test_64bit(std::string vals, double val) {
     return false;
   }
   std::cout << std::hexfloat << result_value << " == " << val << std::endl;
-
   return true;
 }
 bool basiciss_test_64bit(double val) {
diff --git a/tests/exhaustive32_64.cpp b/tests/exhaustive32_64.cpp
index bf945de..95cf6ca 100644
--- a/tests/exhaustive32_64.cpp
+++ b/tests/exhaustive32_64.cpp
@@ -32,6 +32,7 @@ void all_32bit_values() {
         abort();
       }
       if(copysign(1,result_value) != copysign(1,v)) {
+        std::cerr << buffer << std::endl;
         std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << v
               << std::endl;
         abort();
diff --git a/tests/exhaustive32_midpoint.cpp b/tests/exhaustive32_midpoint.cpp
index e55659f..de73248 100644
--- a/tests/exhaustive32_midpoint.cpp
+++ b/tests/exhaustive32_midpoint.cpp
@@ -76,6 +76,7 @@ void allvalues() {
         abort();
       }
       if(copysign(1,result_value) != copysign(1,v)) {
+        std::cerr << buffer << std::endl;
         std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << v
               << std::endl;
         abort();
diff --git a/tests/long_exhaustive32.cpp b/tests/long_exhaustive32.cpp
index c0b6a16..916960c 100644
--- a/tests/long_exhaustive32.cpp
+++ b/tests/long_exhaustive32.cpp
@@ -31,6 +31,7 @@ void allvalues() {
         abort();
       }
       if(copysign(1,result_value) != copysign(1,v)) {
+        std::cerr << buffer << std::endl;
         std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << v
               << std::endl;
         abort();
diff --git a/tests/long_random64.cpp b/tests/long_random64.cpp
index b07d7a1..b8a6b05 100644
--- a/tests/long_random64.cpp
+++ b/tests/long_random64.cpp
@@ -61,6 +61,7 @@ void random_values(size_t N) {
         }
       }
       if(copysign(1,result_value) != copysign(1,v)) {
+        std::cerr << buffer << std::endl;
         std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << v
               << std::endl;
         abort();
diff --git a/tests/random64.cpp b/tests/random64.cpp
index 45f3746..3f37975 100644
--- a/tests/random64.cpp
+++ b/tests/random64.cpp
@@ -63,6 +63,7 @@ void random_values(size_t N) {
         }
       }
       if(copysign(1,result_value) != copysign(1,v)) {
+        std::cerr << buffer << std::endl;
         std::cerr << "I got " << std::hexfloat << result_value << " but I was expecting " << v
               << std::endl;
         abort();