From 82e272a109281f750909d1feade784c778d8b592 Mon Sep 17 00:00:00 2001 From: Ryan Ofsky Date: Tue, 11 Oct 2022 23:35:28 -0400 Subject: [PATCH] refactor: Move src/interfaces/*.cpp files to libbitcoin_common.a These belong in libbitcoin_common.a, not libbitcoin_util.a, because they aren't general-purpose utilities, they just contain common code that is used by both the node and the wallet. Another reason to reason to not include these in libbitcoin_util.a is to prevent them from being used by the kernel library. --- src/Makefile.am | 4 +- .../handler.cpp => common/interfaces.cpp} | 40 +++++++++++-------- src/interfaces/echo.cpp | 18 --------- src/interfaces/init.cpp | 17 -------- src/interfaces/init.h | 19 ++++----- 5 files changed, 35 insertions(+), 63 deletions(-) rename src/{interfaces/handler.cpp => common/interfaces.cpp} (63%) delete mode 100644 src/interfaces/echo.cpp delete mode 100644 src/interfaces/init.cpp diff --git a/src/Makefile.am b/src/Makefile.am index 2e2da54b2d..63a690d49f 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -633,6 +633,7 @@ libbitcoin_common_a_SOURCES = \ chainparams.cpp \ coins.cpp \ common/bloom.cpp \ + common/interfaces.cpp \ common/run_command.cpp \ compressor.cpp \ core_read.cpp \ @@ -678,9 +679,6 @@ libbitcoin_util_a_SOURCES = \ chainparamsbase.cpp \ clientversion.cpp \ fs.cpp \ - interfaces/echo.cpp \ - interfaces/handler.cpp \ - interfaces/init.cpp \ logging.cpp \ random.cpp \ randomenv.cpp \ diff --git a/src/interfaces/handler.cpp b/src/common/interfaces.cpp similarity index 63% rename from src/interfaces/handler.cpp rename to src/common/interfaces.cpp index adb7031cbc..289856871d 100644 --- a/src/interfaces/handler.cpp +++ b/src/common/interfaces.cpp @@ -1,17 +1,26 @@ -// Copyright (c) 2018-2021 The Bitcoin Core developers +// Copyright (c) 2021 The Bitcoin Core developers // Distributed under the MIT software license, see the accompanying // file COPYING or http://www.opensource.org/licenses/mit-license.php. +#include #include - #include +#include #include -namespace interfaces { +namespace common { namespace { +class CleanupHandler : public interfaces::Handler +{ +public: + explicit CleanupHandler(std::function cleanup) : m_cleanup(std::move(cleanup)) {} + ~CleanupHandler() override { if (!m_cleanup) return; m_cleanup(); m_cleanup = nullptr; } + void disconnect() override { if (!m_cleanup) return; m_cleanup(); m_cleanup = nullptr; } + std::function m_cleanup; +}; -class HandlerImpl : public Handler +class HandlerImpl : public interfaces::Handler { public: explicit HandlerImpl(boost::signals2::connection connection) : m_connection(std::move(connection)) {} @@ -21,25 +30,24 @@ public: boost::signals2::scoped_connection m_connection; }; -class CleanupHandler : public Handler +class EchoImpl : public interfaces::Echo { public: - explicit CleanupHandler(std::function cleanup) : m_cleanup(std::move(cleanup)) {} - ~CleanupHandler() override { if (!m_cleanup) return; m_cleanup(); m_cleanup = nullptr; } - void disconnect() override { if (!m_cleanup) return; m_cleanup(); m_cleanup = nullptr; } - std::function m_cleanup; + std::string echo(const std::string& echo) override { return echo; } }; - } // namespace +} // namespace common + +namespace interfaces { +std::unique_ptr MakeHandler(std::function cleanup) +{ + return std::make_unique(std::move(cleanup)); +} std::unique_ptr MakeHandler(boost::signals2::connection connection) { - return std::make_unique(std::move(connection)); -} - -std::unique_ptr MakeHandler(std::function cleanup) -{ - return std::make_unique(std::move(cleanup)); + return std::make_unique(std::move(connection)); } +std::unique_ptr MakeEcho() { return std::make_unique(); } } // namespace interfaces diff --git a/src/interfaces/echo.cpp b/src/interfaces/echo.cpp deleted file mode 100644 index 9bbb42217b..0000000000 --- a/src/interfaces/echo.cpp +++ /dev/null @@ -1,18 +0,0 @@ -// Copyright (c) 2021 The Bitcoin Core developers -// Distributed under the MIT software license, see the accompanying -// file COPYING or http://www.opensource.org/licenses/mit-license.php. - -#include - -#include - -namespace interfaces { -namespace { -class EchoImpl : public Echo -{ -public: - std::string echo(const std::string& echo) override { return echo; } -}; -} // namespace -std::unique_ptr MakeEcho() { return std::make_unique(); } -} // namespace interfaces diff --git a/src/interfaces/init.cpp b/src/interfaces/init.cpp deleted file mode 100644 index f0f8aa5fed..0000000000 --- a/src/interfaces/init.cpp +++ /dev/null @@ -1,17 +0,0 @@ -// Copyright (c) 2021 The Bitcoin Core developers -// Distributed under the MIT software license, see the accompanying -// file COPYING or http://www.opensource.org/licenses/mit-license.php. - -#include -#include -#include -#include -#include - -namespace interfaces { -std::unique_ptr Init::makeNode() { return {}; } -std::unique_ptr Init::makeChain() { return {}; } -std::unique_ptr Init::makeWalletLoader(Chain& chain) { return {}; } -std::unique_ptr Init::makeEcho() { return {}; } -Ipc* Init::ipc() { return nullptr; } -} // namespace interfaces diff --git a/src/interfaces/init.h b/src/interfaces/init.h index 2153076366..5b8f61640e 100644 --- a/src/interfaces/init.h +++ b/src/interfaces/init.h @@ -5,6 +5,11 @@ #ifndef BITCOIN_INTERFACES_INIT_H #define BITCOIN_INTERFACES_INIT_H +#include +#include +#include +#include + #include namespace node { @@ -12,11 +17,7 @@ struct NodeContext; } // namespace node namespace interfaces { -class Chain; -class Echo; class Ipc; -class Node; -class WalletLoader; //! Initial interface created when a process is first started, and used to give //! and get access to other interfaces (Node, Chain, Wallet, etc). @@ -29,11 +30,11 @@ class Init { public: virtual ~Init() = default; - virtual std::unique_ptr makeNode(); - virtual std::unique_ptr makeChain(); - virtual std::unique_ptr makeWalletLoader(Chain& chain); - virtual std::unique_ptr makeEcho(); - virtual Ipc* ipc(); + virtual std::unique_ptr makeNode() { return nullptr; } + virtual std::unique_ptr makeChain() { return nullptr; } + virtual std::unique_ptr makeWalletLoader(Chain& chain) { return nullptr; } + virtual std::unique_ptr makeEcho() { return nullptr; } + virtual Ipc* ipc() { return nullptr; } }; //! Return implementation of Init interface for the node process. If the argv