Fix unreachable code warning in tests

include/boost/beast/core/impl/buffers_cat.hpp

@@ -61,13 +61,11 @@ class buffers_cat_view<Buffer>
 #define BOOST_BEAST_LOGIC_ERROR(s) \
     { \
         BOOST_THROW_EXCEPTION(std::logic_error((s))); \
-        BOOST_BEAST_UNREACHABLE(); \
     }
 
 #define BOOST_BEAST_LOGIC_ERROR_RETURN(s, v) \
     { \
         BOOST_THROW_EXCEPTION(std::logic_error((s))); \
-        return v; \
     }
 
 #else
@@ -94,13 +92,6 @@ struct buffers_cat_view_iterator_base
     {
         char unused = 0; // make g++8 happy
 
-        net::mutable_buffer
-        operator*() const
-        {
-            BOOST_BEAST_LOGIC_ERROR_RETURN(
-                    "Dereferencing a one-past-the-end iterator", {});
-        }
-
         operator bool() const noexcept
         {
             return true;
@@ -189,6 +180,13 @@ class buffers_cat_view<Bn...>::const_iterator
                 "Dereferencing a default-constructed iterator", {});
         }
 
+        reference
+        operator()(mp11::mp_size_t<sizeof...(Bn)+1>)
+        {
+            BOOST_BEAST_LOGIC_ERROR_RETURN(
+                "Dereferencing a one-past-the-end iterator", {});
+        }
+
         template<class I>
         reference operator()(I)
         {

test/beast/core/saved_handler.cpp

@@ -7,11 +7,17 @@
 // Official repository: https://github.com/boostorg/beast
 //
 
+#include <boost/config.hpp>
+
+#ifdef BOOST_MSVC
+#pragma warning(push)
+#pragma warning(disable: 4702) // unreachable code
+#endif
+
 // Test that header file is self-contained.
 #include <boost/beast/core/saved_handler.hpp>
 #include <boost/asio/bind_cancellation_slot.hpp>
 #include <boost/beast/_experimental/unit_test/suite.hpp>
-#include <stdexcept>
 
 namespace boost {
 namespace beast {
@@ -220,3 +226,7 @@ BEAST_DEFINE_TESTSUITE(beast,core,saved_handler);
 
 } // beast
 } // boost
+
+#ifdef BOOST_MSVC
+#pragma warning(pop)
+#endif