From c838ce514fe5a264b68e25b988988382302c6dfc Mon Sep 17 00:00:00 2001 From: Vasil Dimov Date: Tue, 23 Jul 2024 12:21:41 +0200 Subject: [PATCH] doc: use proper doxygen formatting for CTxMemPool::cs Having `@par title` followed by an empty line renders improperly in Doxygen - it results in a paragraph with a title but without a body. https://www.doxygen.nl/manual/commands.html#cmdpar This also results in a compiler warning (or error) with Clang 19: ``` ./txmempool.h:368:34: error: empty paragraph passed to '@par' command [-Werror,-Wdocumentation] 368 | * @par Consistency guarantees | ~~~~~~~~~~~~~~~~~~~~~~~~~~^ 1 error generated. ``` Github-Pull: #30504 Rebased-From: 6a5e9e40e1dd3d397020703feb9aa0b6f4577c98 --- src/txmempool.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/txmempool.h b/src/txmempool.h index b98355c65f1..804d4807a1c 100644 --- a/src/txmempool.h +++ b/src/txmempool.h @@ -363,9 +363,7 @@ public: * that are guarded by it. * * @par Consistency guarantees - * * By design, it is guaranteed that: - * * 1. Locking both `cs_main` and `mempool.cs` will give a view of mempool * that is consistent with current chain tip (`ActiveChain()` and * `CoinsTip()`) and is fully populated. Fully populated means that if the @@ -373,7 +371,6 @@ public: * previously active chain, all the missing transactions will have been * re-added to the mempool and should be present if they meet size and * consistency constraints. - * * 2. Locking `mempool.cs` without `cs_main` will give a view of a mempool * consistent with some chain that was active since `cs_main` was last * locked, and that is fully populated as described above. It is ok for