[doc] assert that default min relay feerate and incremental are the same
What the developer wrote
Authored by glozow
[doc] assert that default min relay feerate and incremental are the same
What changed, and why it matters
This commit adds a compile-time check (static_assert) to ensure two internal fee constants are equal. It is a documentation/safety assertion with no functional code change and no security impact on its own.
No action required. Treat as non-security documentation/development-safety commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch inserts static_assert(DEFAULT_MIN_RELAY_TX_FEE == DEFAULT_INCREMENTAL_RELAY_FEE); in ApplyArgsManOptions. This is a compile-time assertion verifying that two default feerate constants remain equal. It does not alter runtime behavior, parsing, validation, or network logic.
Changed components
src/node/mempool_args.cppInspect captured patch +1 / −0
diff --git a/src/node/mempool_args.cpp b/src/node/mempool_args.cpp
index 11c77ff5..abbe97d9 100644
--- a/src/node/mempool_args.cpp
+++ b/src/node/mempool_args.cpp
@@ -65,6 +65,7 @@ util::Result<void> ApplyArgsManOptions(const ArgsManager& argsman, const CChainP
}
}
+ static_assert(DEFAULT_MIN_RELAY_TX_FEE == DEFAULT_INCREMENTAL_RELAY_FEE);
if (const auto arg{argsman.GetArg("-minrelaytxfee")}) {
if (std::optional<CAmount> min_relay_feerate = ParseMoney(*arg)) {
// High fee check is done afterward in CWallet::Create()
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.