Correct implementation of std::error_code wrapper.

OrchidTechnologies · Jul 21, 2024 · 935542b · 935542b
1 parent e71727d
commit 935542b
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 25 deletions.
diff --git a/lib-protocol/source/adapter.hpp b/lib-protocol/source/adapter.hpp
@@ -60,8 +60,8 @@ class Adapter {
     U<Stream> stream_;
 
     template <typename Handler_>
-    void Convert(Handler_ &&handler, const std::exception_ptr &error) {
-        boost::asio::post(get_executor(), [handler = std::forward<Handler_>(handler), error = Category::Convert(error)]() mutable {
+    void Convert(Handler_ &&handler) {
+        boost::asio::post(get_executor(), [handler = std::forward<Handler_>(handler), error = Category::Convert(std::current_exception())]() mutable {
             std::move(handler)(error, 0);
         });
     }
@@ -98,7 +98,7 @@ class Adapter {
                     return std::move(handler)(eof ? asio::error::eof : boost::system::error_code(), writ);
                 });
             } catch (...) {
-                Convert(std::move(handler), std::current_exception());
+                Convert(std::move(handler));
             }
         }, __FUNCTION__);
     }
@@ -118,7 +118,7 @@ class Adapter {
                     return std::move(handler)(boost::system::error_code(), writ);
                 });
             } catch (...) {
-                Convert(std::move(handler), std::current_exception());
+                Convert(std::move(handler));
             }
         }, __FUNCTION__);
     }

diff --git a/lib-shared/source/category.cpp → lib-protocol/source/category.cpp b/lib-shared/source/category.cpp → lib-protocol/source/category.cpp
@@ -38,19 +38,15 @@ namespace {
 
 std::string Category::message(int index) const {
     try {
-        std::rethrow_exception(Convert(index));
+        const std::unique_lock<std::mutex> lock(mutex_);
+        if (auto error = errors_.extract(index))
+            std::rethrow_exception(error.mapped());
+        orc_insist(false);
     } catch (const std::exception &error) {
         return error.what();
     }
 }
 
-std::exception_ptr Category::Convert(int index) noexcept {
-    const std::unique_lock<std::mutex> lock(mutex_);
-    if (const auto error = errors_.extract(index))
-        return error.mapped();
-    orc_insist(false);
-}
-
 boost::system::error_code Category::Convert(const std::exception_ptr &error) noexcept {
     const std::unique_lock<std::mutex> lock(mutex_);
     // XXX: clang-tidy might need fixing, as this just bans ?:

diff --git a/lib-shared/source/category.hpp → lib-protocol/source/category.hpp b/lib-shared/source/category.hpp → lib-protocol/source/category.hpp
@@ -37,8 +37,6 @@ class Category :
     }
 
     std::string message(int index) const override;
-
-    static std::exception_ptr Convert(int index) noexcept;
     static boost::system::error_code Convert(const std::exception_ptr &error) noexcept;
 };
 
@@ -53,4 +51,8 @@ constexpr const boost::system::error_category &orchid_category() noexcept {
     return orc::Categories<void>::category_;
 }
 
+#define orc_adapt(error) do { \
+    orc_throw(error.what()); \
+} while (false)
+
 #endif//ORCHID_CATEGORY_HPP
diff --git a/lib-protocol/source/connection.hpp b/lib-protocol/source/connection.hpp
@@ -81,12 +81,10 @@ class Association :
         co_return writ;
     }
 
-    task<void> Open(const Socket &endpoint) { orc_ahead orc_block({ try {
+    task<void> Open(const Socket &endpoint) { orc_ahead orc_block({
         co_await association_.async_connect(endpoint, Adapt());
         association_.non_blocking(true);
-    } catch (const asio::system_error &error) {
-        orc_adapt(error);
-    } }, "connecting to " << endpoint); }
+    }, "connecting to " << endpoint); }
 
     void Shut() noexcept override {
         association_.close();

diff --git a/lib-shared/source/error.hpp b/lib-shared/source/error.hpp
@@ -70,13 +70,6 @@ class Error final :
     throw orc_log(orc::Error(), text); \
 } while (false)
 
-#define orc_adapt(error) do { \
-    const auto code(error.code()); \
-    if (code.category() == orchid_category()) \
-        std::rethrow_exception(Category::Convert(code.value())); \
-    orc_throw(code.message() << " (" << code << ")"); \
-} while (false)
-
 #define orc_assert_(code, text) do { \
     if ((code)) break; \
     orc_throw(text); \