From 07db391c46c7258edbb0a60d0272056566e6f762 Mon Sep 17 00:00:00 2001 From: Bart <11445373+bthomee@users.noreply.github.com> Date: Fri, 24 Jul 2026 11:39:00 -0400 Subject: [PATCH] Clang-tidy changes --- src/xrpld/overlay/detail/PeerImp.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/xrpld/overlay/detail/PeerImp.cpp b/src/xrpld/overlay/detail/PeerImp.cpp index 2b0ba013bd..e1b6e502fb 100644 --- a/src/xrpld/overlay/detail/PeerImp.cpp +++ b/src/xrpld/overlay/detail/PeerImp.cpp @@ -1536,13 +1536,16 @@ PeerImp::onMessage(std::shared_ptr const& m) } // Charge once per request. Requesting too many node IDs is charged even for a relay - // response, since it's a distinct infraction from the "received a get ledger request" - // charge below, which is skipped for relay responses. + // response, since it's a distinct infraction from the "get ledger request" charge below, + // which is skipped for relay responses. if (tooManyNodeIds) + { peer->charge(Resource::kFeeModerateBurdenPeer, "TMGetLedger: too many node IDs"); + } else if (!m->has_requestcookie()) - peer->charge( - Resource::kFeeModerateBurdenPeer, "TMGetLedger: received get ledger request"); + { + peer->charge(Resource::kFeeModerateBurdenPeer, "TMGetLedger: get ledger request"); + } peer->processLedgerRequest(m, std::move(nodeIDs)); });