refactor: add util::to_string(bool) for convenience

Author: kwvg

src/coinjoin/coinjoin.cpp

@@ -7,6 +7,7 @@
 #include <bls/bls.h>
 #include <chainlock/chainlock.h>
 #include <instantsend/instantsend.h>
+#include <util/helpers.h>
 
 #include <chain.h>
 #include <chainparams.h>
@@ -63,7 +64,7 @@ bool CCoinJoinQueue::IsTimeOutOfBounds(int64_t current_time) const
 [[nodiscard]] std::string CCoinJoinQueue::ToString() const
 {
     return strprintf("nDenom=%d, nTime=%lld, fReady=%s, fTried=%s, masternode=%s",
-        nDenom, nTime, fReady ? "true" : "false", fTried ? "true" : "false", masternodeOutpoint.ToStringShort());
+        nDenom, nTime, util::to_string(fReady), util::to_string(fTried), masternodeOutpoint.ToStringShort());
 }
 
 uint256 CCoinJoinBroadcastTx::GetSignatureHash() const

src/llmq/quorumsman.cpp

@@ -285,7 +285,7 @@ std::vector<CQuorumCPtr> CQuorumManager::ScanQuorums(Consensus::LLMQType llmqTyp
         if (!quorum) {
             LogPrintf("%s: ERROR! Unexpected missing quorum with llmqType=%d, blockHash=%s, populate_cache=%s\n",
                       __func__, std23::to_underlying(llmqType), pQuorumBaseBlockIndex->GetBlockHash().ToString(),
-                      populate_cache ? "true" : "false");
+                      util::to_string(populate_cache));
             return {};
         }
         vecResultQuorums.emplace_back(quorum);

src/qt/guiutil_font.cpp

@@ -4,6 +4,8 @@
 
 #include <qt/guiutil_font.h>
 
+#include <util/helpers.h>
+
 #include <tinyformat.h>
 #include <util/std23.h>
 #include <util/system.h>
@@ -186,7 +188,7 @@ QFont getFont(const GUIUtil::FontAttrib& font_attrib)
     if (gArgs.GetBoolArg("-debug-ui", false)) {
         qDebug() << qstrprintf("%s: font size: %d, family: %s, style: %s, weight: %d match %s", __func__,
                                font.pointSizeF(), font.family().toStdString(), font.styleName().toStdString(),
-                               font.weight(), font.exactMatch() ? "true" : "false");
+                               font.weight(), util::to_string(font.exactMatch()));
     }
 
     return font;
@@ -533,7 +535,7 @@ void setApplicationFont()
 
     qDebug() << qstrprintf("%s: %s family: %s, style: %s match: %s", __func__, qApp->font().toString().toStdString(),
                            qApp->font().family().toStdString(), qApp->font().styleName().toStdString(),
-                           qApp->font().exactMatch() ? "true" : "false");
+                           util::to_string(qApp->font().exactMatch()));
 }
 
 void setFont(const std::vector<QWidget*>& vecWidgets, const FontAttrib& font_attrib)

src/rpc/mempool.cpp

@@ -5,6 +5,10 @@
 
 #include <rpc/blockchain.h>
 
+#include <instantsend/instantsend.h>
+#include <llmq/context.h>
+#include <util/helpers.h>
+
 #include <chainparams.h>
 #include <core_io.h>
 #include <fs.h>
@@ -20,9 +24,6 @@
 #include <util/system.h>
 #include <util/time.h>
 
-#include <instantsend/instantsend.h>
-#include <llmq/context.h>
-
 using node::DEFAULT_MAX_RAW_TX_FEE_RATE;
 using node::NodeContext;
 
@@ -326,7 +327,7 @@ static void entryToJSON(const CTxMemPool& pool, UniValue& info, const CTxMemPool
     }
 
     info.pushKV("spentby", spent);
-    info.pushKV("instantlock", isman ? (isman->IsLocked(tx.GetHash()) ? "true" : "false") : "unknown");
+    info.pushKV("instantlock", isman ? util::to_string(isman->IsLocked(tx.GetHash())) : "unknown");
     info.pushKV("unbroadcast", pool.IsUnbroadcastTx(tx.GetHash()));
 }
 

src/util/helpers.h

@@ -8,6 +8,7 @@
 #include <memory>
 #include <optional>
 #include <ranges>
+#include <string_view>
 
 namespace util {
 template <typename X, typename Z>
@@ -43,6 +44,11 @@ inline bool shared_ptr_not_equal(const std::shared_ptr<T>& lhs, const std::share
     if (!lhs || !rhs) return true;  // Inequal initialization state
     return *lhs != *rhs;            // Deep comparison
 }
+
+inline constexpr std::string_view to_string(bool value)
+{
+    return value ? "true" : "false";
+}
 } // namespace util
 
 #endif // BITCOIN_UTIL_HELPERS_H