Create separate header for namespace level typedefs.

Author: evoskuil

Makefile.am

@@ -144,6 +144,7 @@ include_bitcoin_database_HEADERS = \
     include/bitcoin/database/query.hpp \
     include/bitcoin/database/settings.hpp \
     include/bitcoin/database/store.hpp \
+    include/bitcoin/database/types.hpp \
     include/bitcoin/database/version.hpp
 
 include_bitcoin_database_filedir = ${includedir}/bitcoin/database/file

builds/msvc/vs2022/libbitcoin-database/libbitcoin-database.vcxproj

@@ -197,6 +197,7 @@
     <ClInclude Include="..\..\..\..\include\bitcoin\database\tables\states.hpp" />
     <ClInclude Include="..\..\..\..\include\bitcoin\database\tables\table.hpp" />
     <ClInclude Include="..\..\..\..\include\bitcoin\database\tables\tables.hpp" />
+    <ClInclude Include="..\..\..\..\include\bitcoin\database\types.hpp" />
     <ClInclude Include="..\..\..\..\include\bitcoin\database\version.hpp" />
     <ClInclude Include="..\..\..\..\src\memory\mman-win32\mman.hpp" />
   </ItemGroup>

builds/msvc/vs2022/libbitcoin-database/libbitcoin-database.vcxproj.filters

@@ -290,6 +290,9 @@
     <ClInclude Include="..\..\..\..\include\bitcoin\database\tables\tables.hpp">
       <Filter>include\bitcoin\database\tables</Filter>
     </ClInclude>
+    <ClInclude Include="..\..\..\..\include\bitcoin\database\types.hpp">
+      <Filter>include\bitcoin\database</Filter>
+    </ClInclude>
     <ClInclude Include="..\..\..\..\include\bitcoin\database\version.hpp">
       <Filter>include\bitcoin\database</Filter>
     </ClInclude>

include/bitcoin/database.hpp

@@ -21,6 +21,7 @@
 #include <bitcoin/database/query.hpp>
 #include <bitcoin/database/settings.hpp>
 #include <bitcoin/database/store.hpp>
+#include <bitcoin/database/types.hpp>
 #include <bitcoin/database/version.hpp>
 #include <bitcoin/database/file/file.hpp>
 #include <bitcoin/database/file/rotator.hpp>

include/bitcoin/database/impl/query/consensus.ipp

@@ -185,7 +185,7 @@ bool CLASS::is_strong(const tx_link& tx) const NOEXCEPT
 
 // protected
 TEMPLATE
-error::error_t CLASS::unspendable(uint32_t sequence, bool coinbase,
+code CLASS::unspendable(uint32_t sequence, bool coinbase,
     const tx_link& tx, uint32_t version, const context& ctx) const NOEXCEPT
 {
     // Ensure prevout tx is in a strong block, first try self link.
@@ -317,27 +317,27 @@ code CLASS::block_confirmable(const header_link& link) const NOEXCEPT
 
     // Get points for each tx, and sum the total number.
     std::transform(parallel, txs.begin(), txs.end(), sets.begin(), to_set);
-    if (failure)
-        return { failure.load() };
+    if (failure != error::success)
+        return failure;
 
     // Check double spends strength, populates prevout parent tx/cb/sq links.
     if ((ec = populate_prevouts(sets, points, link)))
         return ec;
 
-    const auto is_unspendable = [this, &ctx, &failure](const auto& set) NOEXCEPT
+    const bool is_spendable = [this, &ctx, &failure](const auto& set) NOEXCEPT
     {
         error::error_t ec{};
         for (const auto& point: set.points)
             if (!point.tx.is_terminal() && ((ec = unspendable(point.sequence,
                 point.coinbase, point.tx, set.version, ctx))))
                 failure.store(ec);
 
-        return failure != error::success;
+        return failure == error::success;
     };
 
     // Check all spends for spendability (strong, unlocked and mature).
-    if (std::any_of(parallel, sets.begin(), sets.end(), is_unspendable))
-        return { failure.load() };
+    if (!std::all_of(parallel, sets.begin(), sets.end(), is_spendable))
+        return failure;
 
     return error::success;
 }

include/bitcoin/database/query.hpp

@@ -24,36 +24,11 @@
 #include <utility>
 #include <bitcoin/system.hpp>
 #include <bitcoin/database/define.hpp>
-#include <bitcoin/database/error.hpp>
-#include <bitcoin/database/primitives/primitives.hpp>
-#include <bitcoin/database/tables/tables.hpp>
+#include <bitcoin/database/types.hpp>
 
 namespace libbitcoin {
 namespace database {
 
-/// Database type aliases.
-using height_link = table::height::link;
-using header_link = table::header::link;
-using output_link = table::output::link;
-using input_link = table::input::link;
-using outs_link = table::outs::link;
-using ins_link = table::ins::link;
-using point_link = table::point::link;
-using tx_link = table::transaction::link;
-using filter_link = table::filter_tx::link;
-using strong_link = table::strong_tx::link;
-
-using header_links = std::vector<header_link::integer>;
-using tx_links = std::vector<tx_link::integer>;
-using input_links = std::vector<input_link::integer>;
-using output_links = std::vector<output_link::integer>;
-using point_links = std::vector<point_link::integer>;
-using two_counts = std::pair<size_t, size_t>;
-using point_key = table::point::key;
-
-struct header_state{ header_link link; code ec; };
-using header_states = std::vector<header_state>;
-
 // Writers (non-const) are only: push_, pop_, set_ and initialize.
 template <typename Store>
 class query
@@ -652,7 +627,7 @@ class query
         const context& ctx) const NOEXCEPT;
 
     /// Called by block_confirmable (populate and check double spends).
-    error::error_t unspendable(uint32_t sequence, bool coinbase,
+    code unspendable(uint32_t sequence, bool coinbase,
         const tx_link& prevout_tx, uint32_t version,
         const context& ctx) const NOEXCEPT;
 

include/bitcoin/database/types.hpp

@@ -0,0 +1,59 @@
+/**
+ * Copyright (c) 2011-2025 libbitcoin developers (see AUTHORS)
+ *
+ * This file is part of libbitcoin.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+#ifndef LIBBITCOIN_DATABASE_TYPES_HPP
+#define LIBBITCOIN_DATABASE_TYPES_HPP
+
+#include <set>
+#include <utility>
+#include <bitcoin/database/define.hpp>
+#include <bitcoin/database/tables/tables.hpp>
+
+namespace libbitcoin {
+namespace database {
+
+/// Database type aliases.
+using height_link = table::height::link;
+using header_link = table::header::link;
+using output_link = table::output::link;
+using input_link = table::input::link;
+using outs_link = table::outs::link;
+using ins_link = table::ins::link;
+using point_link = table::point::link;
+using tx_link = table::transaction::link;
+using filter_link = table::filter_tx::link;
+using strong_link = table::strong_tx::link;
+
+using header_links = std::vector<header_link::integer>;
+using tx_links = std::vector<tx_link::integer>;
+using input_links = std::vector<input_link::integer>;
+using output_links = std::vector<output_link::integer>;
+using point_links = std::vector<point_link::integer>;
+using two_counts = std::pair<size_t, size_t>;
+using point_key = table::point::key;
+
+struct header_state{ header_link link; code ec; };
+using header_states = std::vector<header_state>;
+
+using inpoint_set = std::set<system::chain::point>;
+using outpoint_set = std::set<system::chain::outpoint>;
+
+} // namespace database
+} // namespace libbitcoin
+
+#endif