correct rounding

fix memory leak for constructor failure in load_construct_data
fix another bug in loading pointers

[SVN r86487]

test/A.cpp

@@ -148,13 +148,13 @@ bool A::operator==(const A &rhs) const
         return false; 
     if(v != rhs.v)
         return false; 
-    if(w == 0 && std::fabs(rhs.w) > std::numeric_limits<float>::epsilon())
+    if(w == 0 && std::fabs(rhs.w) > 2 * std::numeric_limits<float>::round_error())
         return false;
-    if(std::fabs(rhs.w/w - 1.0) > std::numeric_limits<float>::epsilon())
+    if(std::fabs(rhs.w/w - 1.0) > 2 * std::numeric_limits<float>::round_error())
         return false;
-    if(x == 0 && std::fabs(rhs.x - x) > std::numeric_limits<float>::epsilon())
+    if(x == 0 && std::fabs(rhs.x - x) > 2 * std::numeric_limits<double>::round_error())
         return false;
-    if(std::fabs(rhs.x/x - 1.0) > std::numeric_limits<float>::epsilon())
+    if(std::fabs(rhs.x/x - 1.0) > 2 * std::numeric_limits<double>::round_error())
         return false;
     if(0 != y.compare(rhs.y))
         return false;

test/test_codecvt_null.cpp

@@ -10,7 +10,7 @@
 // which use wchar_t as 2 byte objects will emit warnings.  These should be
 // ignored.
 
-#include <algorithm>
+#include <algorithm> // std::copy
 #include <fstream>
 #include <iostream>
 #include <iterator>

test/test_complex.cpp

@@ -68,8 +68,8 @@ int test_main( int /* argc */, char* /* argv */[] )
         ia >> boost::serialization::make_nvp("adoublecomplex", b1);
     }
 
-    BOOST_CHECK(std::abs(a-a1) <= 2.*std::numeric_limits<float>::round_error());
-    BOOST_CHECK(std::abs(b-b1) <= 2.*std::numeric_limits<double>::round_error());
+    BOOST_CHECK(std::abs(a-a1) <= (2 * std::numeric_limits<float>::round_error()));
+    BOOST_CHECK(std::abs(b-b1) <= (2 * std::numeric_limits<double>::round_error()));
 
     std::remove(testfile);
     return EXIT_SUCCESS;

test/test_iterators.cpp

@@ -6,7 +6,7 @@
 // License, Version 1.0. (See accompanying file LICENSE_1_0.txt or copy at
 // http://www.boost.org/LICENSE_1_0.txt)
 
-#include <algorithm>
+#include <algorithm> // std::copy
 #include <vector>
 #include <cstdlib> // for rand
 #include <functional>

test/test_map.cpp

@@ -8,7 +8,7 @@
 
 // should pass compilation and execution
 
-#include <algorithm>
+#include <algorithm> // std::copy
 #include <vector>
 #include <fstream>
 #include <cstddef> // size_t, NULL

test/test_set.cpp

@@ -12,7 +12,7 @@
 #include <cstdio> // remove
 #include <fstream>
 
-#include <algorithm>
+#include <algorithm> // std::copy
 #include <vector>
 
 #include <boost/config.hpp>

test/test_simple_class.cpp

@@ -52,16 +52,16 @@ bool A::check_equal(const A &rhs) const
     BOOST_CHECK_EQUAL(l, rhs.l);
     BOOST_CHECK(!(
         w == 0 
-        && std::fabs(rhs.w) > std::numeric_limits<float>::epsilon()
+        && std::fabs(rhs.w) > std::numeric_limits<float>::round_error()
     ));
     BOOST_CHECK(!(
-        std::fabs(rhs.w/w - 1.0) > std::numeric_limits<float>::epsilon()
+        std::fabs(rhs.w/w - 1.0) > 2.0 * std::numeric_limits<float>::round_error()
     ));
     BOOST_CHECK(!(
-        x == 0 && std::fabs(rhs.x - x) > std::numeric_limits<float>::epsilon()
+        x == 0 && std::fabs(rhs.x - x) > 2.0 * std::numeric_limits<double>::round_error()
     ));
     BOOST_CHECK(!(
-        std::fabs(rhs.x/x - 1.0) > std::numeric_limits<float>::epsilon()
+        std::fabs(rhs.x/x - 1.0) > 2.0 * std::numeric_limits<double>::round_error()
     ));
     BOOST_CHECK(!(0 != y.compare(rhs.y)));
     #ifndef BOOST_NO_STD_WSTRING

test/test_utf8_codecvt.cpp

@@ -6,7 +6,7 @@
 // License, Version 1.0. (See accompanying file LICENSE_1_0.txt or copy at
 // http://www.boost.org/LICENSE_1_0.txt)
 
-#include <algorithm>
+#include <algorithm> // std::copy
 #include <fstream>
 #include <iostream>
 #include <iterator>