From 18c6f81634de470e20053ef6e5132ff14fce7702 Mon Sep 17 00:00:00 2001
From: Mohammad Nejati <ashtumashtum@gmail.com>
Date: Fri, 25 Oct 2024 09:37:27 +0000
Subject: [PATCH] Fix unreachable code warning in tests

---
 include/boost/beast/core/impl/buffers_cat.hpp | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/include/boost/beast/core/impl/buffers_cat.hpp b/include/boost/beast/core/impl/buffers_cat.hpp
index 35d1a34d1e..98c4775852 100644
--- a/include/boost/beast/core/impl/buffers_cat.hpp
+++ b/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)
         {