From be82139b2a1e5bcd7c47391672d38fa5aa05bc0b Mon Sep 17 00:00:00 2001
From: Jon Atack <jon@atack.com>
Date: Sun, 16 Feb 2025 09:24:12 -0600
Subject: [PATCH] cli, refactor: simplify DetailsRequested()

The bounds check is no longer needed after the merge of PR 21192.
---
 src/bitcoin-cli.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bitcoin-cli.cpp b/src/bitcoin-cli.cpp
index 75911d00874..44198b9a9be 100644
--- a/src/bitcoin-cli.cpp
+++ b/src/bitcoin-cli.cpp
@@ -404,7 +404,7 @@ private:
         return UNKNOWN_NETWORK;
     }
     uint8_t m_details_level{0}; //!< Optional user-supplied arg to set dashboard details level
-    bool DetailsRequested() const { return m_details_level > 0 && m_details_level < 5; }
+    bool DetailsRequested() const { return m_details_level; }
     bool IsAddressSelected() const { return m_details_level == 2 || m_details_level == 4; }
     bool IsVersionSelected() const { return m_details_level == 3 || m_details_level == 4; }
     bool m_outbound_only_selected{false};