Handle compiler warnings and cover-misses

Author: ckormanyos

test/link_1.cpp

@@ -6,6 +6,7 @@
 
 void test_odr_use(const boost::decimal::rounding_mode*);
 
+// LCOV_EXCL_START
 template <typename T>
 void test()
 {
@@ -18,3 +19,4 @@ void f1()
     test<boost::decimal::decimal64>();
     test<boost::decimal::decimal128>();
 }
+// LCOV_EXCL_STOP

test/link_2.cpp

@@ -6,6 +6,7 @@
 
 void test_odr_use(const boost::decimal::rounding_mode*);
 
+// LCOV_EXCL_START
 template <typename T>
 void test()
 {
@@ -18,3 +19,4 @@ void f2()
     test<boost::decimal::decimal64>();
     test<boost::decimal::decimal128>();
 }
+// LCOV_EXCL_STOP

test/link_3.cpp

@@ -4,6 +4,7 @@
 
 #include <boost/decimal.hpp>
 
+// LCOV_EXCL_START
 void f1();
 void f2();
 
@@ -16,3 +17,4 @@ int main()
 void test_odr_use(const boost::decimal::rounding_mode*)
 {
 }
+// LCOV_EXCL_STOP

test/test_acosh.cpp

@@ -56,7 +56,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {

test/test_asinh.cpp

@@ -22,7 +22,6 @@
 #include <boost/core/lightweight_test.hpp>
 
 auto my_zero() -> boost::decimal::decimal32&;
-auto my_one () -> boost::decimal::decimal32&;
 
 namespace local
 {
@@ -56,7 +55,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {
@@ -268,4 +267,3 @@ auto main() -> int
 }
 
 auto my_zero() -> boost::decimal::decimal32& { static boost::decimal::decimal32 val_zero { 0, 0 }; return val_zero; }
-auto my_one () -> boost::decimal::decimal32& { static boost::decimal::decimal32 val_one  { 1, 0 }; return val_one; }

test/test_atanh.cpp

@@ -56,7 +56,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {

test/test_edges_and_behave.cpp

@@ -62,7 +62,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {

test/test_expm1.cpp

@@ -55,7 +55,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {

test/test_frexp_ldexp.cpp

@@ -53,7 +53,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {

test/test_pow.cpp

@@ -58,7 +58,7 @@ namespace local
 
     if(b == static_cast<NumericType>(0))
     {
-      result_is_ok = (fabs(a - b) < tol);
+      result_is_ok = (fabs(a - b) < tol); // LCOV_EXCL_LINE
     }
     else
     {
@@ -324,9 +324,9 @@ namespace local
         decimal_type { 1, 25},
       }};
 
-      for(auto i = 0; i < std::tuple_size<ctrl_val_array_type>::value; ++i)
+      for(auto i = static_cast<std::size_t>(UINT8_C(0)); i < std::tuple_size<ctrl_val_array_type>::value; ++i)
       {
-        const int n = i + 1;
+        const int n = static_cast<int>(i + static_cast<std::size_t>(UINT8_C(1)));
 
         decimal_type p10 = pow(pow(::my_ten<decimal_type>(), n), n);
 
@@ -346,9 +346,9 @@ namespace local
         decimal_type { 1, -25},
       }};
 
-      for(auto i = 0; i < std::tuple_size<ctrl_val_array_type>::value; ++i)
+      for(auto i = static_cast<std::size_t>(UINT8_C(0)); i < std::tuple_size<ctrl_val_array_type>::value; ++i)
       {
-        const int n = i + 1;
+        const int n = static_cast<int>(i + static_cast<std::size_t>(UINT8_C(1)));
 
         decimal_type p10 = pow(pow(::my_ten<decimal_type>(), -n), n);