explcitly fail on errors during parsing in benchmarks

boostorg · Jul 10, 2024 · 96cb8e3 · 96cb8e3
1 parent 393a633
commit 96cb8e3
Showing 1 changed file with 25 additions and 4 deletions.
diff --git a/bench/bench.cpp b/bench/bench.cpp
@@ -304,6 +304,8 @@ class boost_default_impl : public any_impl
                 p.finish(ec);
             if(! ec)
                 auto jv = p.release();
+            else
+                throw system::system_error( ec );
         }
         return clock_type::now() - start;
     }
@@ -343,6 +345,8 @@ class boost_default_impl : public any_impl
 
             if(! ec)
                 auto jv = p.release();
+            else
+                throw system::system_error( ec );
 
             fclose(f);
         }
@@ -414,10 +418,12 @@ class boost_pool_impl : public any_impl
             p.reset(&mr);
             system::error_code ec;
             p.write(s.data(), s.size(), ec);
-            if(! ec)
+            if( !ec.failed() )
                 p.finish(ec);
-            if(! ec)
+            if( !ec.failed() )
                 auto jv = p.release();
+            else
+                throw system::system_error( ec );
         }
         return clock_type::now() - start;
     }
@@ -458,6 +464,8 @@ class boost_pool_impl : public any_impl
 
             if(! ec)
                 auto jv = p.release();
+            else
+                throw system::system_error( ec );
 
             fclose(f);
         }
@@ -601,7 +609,8 @@ class boost_null_impl : public any_impl
             p.reset();
             system::error_code ec;
             p.write(s.data(), s.size(), ec);
-            BOOST_ASSERT(! ec);
+            if( ec.failed() )
+                throw system::system_error( ec );
         }
         return clock_type::now() - start;
     }
@@ -636,8 +645,10 @@ class boost_null_impl : public any_impl
                     break;
             }
 
-            BOOST_ASSERT(! ec);
             fclose(f);
+
+            if( ec.failed() )
+                throw system::system_error( ec );
         }
         return clock_type::now() - start;
     }
@@ -683,6 +694,8 @@ class boost_simple_impl : public any_impl
             monotonic_resource mr;
             auto jv = json::parse(s, ec, &mr, popts);
             (void)jv;
+            if( ec.failed() )
+                throw system::system_error( ec );
         }
         return clock_type::now() - start;
     }
@@ -698,6 +711,8 @@ class boost_simple_impl : public any_impl
             monotonic_resource mr;
             auto jv = json::parse(is, ec, &mr, popts);
             (void)jv;
+            if( ec.failed() )
+                throw system::system_error( ec );
         }
         return clock_type::now() - start;
     }
@@ -749,6 +764,8 @@ class boost_operator_impl : public any_impl
             value jv(&mr);
             is.seekg(0);
             is >> popts >> jv;
+            if( is.fail() )
+                throw system::system_error( std::io_errc::stream );
         }
         return clock_type::now() - start;
     }
@@ -763,6 +780,8 @@ class boost_operator_impl : public any_impl
             value jv(&mr);
             std::ifstream is( fi.name, std::ios::in | std::ios::binary );
             is >> popts >> jv;
+            if( is.fail() )
+                throw system::system_error( std::io_errc::stream );
         }
         return clock_type::now() - start;
     }
@@ -781,6 +800,8 @@ class boost_operator_impl : public any_impl
             std::ostringstream os;
             os << jv;
             out = os.str();
+            if( os.fail() )
+                throw system::system_error( std::io_errc::stream );
         }
         return clock_type::now() - start;
     }