From dbcd040180cfc138e7f5a9b386a3cf1852615aac Mon Sep 17 00:00:00 2001 From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> Date: Wed, 29 Apr 2026 17:31:23 +0100 Subject: [PATCH] fix(telemetry): fix Clang unused-variable and incomplete-type errors - Add [[maybe_unused]] to RAII spans in TxQ.cpp - Include Telemetry.h in RCLConsensus.cpp for complete type Co-Authored-By: Claude Opus 4.6 --- src/xrpld/app/consensus/RCLConsensus.cpp | 1 + src/xrpld/app/misc/detail/TxQ.cpp | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/xrpld/app/consensus/RCLConsensus.cpp b/src/xrpld/app/consensus/RCLConsensus.cpp index a09409ee64..dffdc9c8bc 100644 --- a/src/xrpld/app/consensus/RCLConsensus.cpp +++ b/src/xrpld/app/consensus/RCLConsensus.cpp @@ -63,6 +63,7 @@ #include #include #include +#include #include #include diff --git a/src/xrpld/app/misc/detail/TxQ.cpp b/src/xrpld/app/misc/detail/TxQ.cpp index 32842ab9ad..484e14bed2 100644 --- a/src/xrpld/app/misc/detail/TxQ.cpp +++ b/src/xrpld/app/misc/detail/TxQ.cpp @@ -532,7 +532,7 @@ TxQ::tryClearAccountQueueUpThruTx( beast::Journal j) { using namespace telemetry; - auto span = SpanGuard::span( + [[maybe_unused]] auto span = SpanGuard::span( TraceCategory::Transactions, txq_span::prefix::txq, txq_span::op::batchClear); SeqProxy const tSeqProx{tx.getSeqProxy()}; @@ -1681,7 +1681,7 @@ TxQ::tryDirectApply( beast::Journal j) { using namespace telemetry; - auto span = SpanGuard::span( + [[maybe_unused]] auto span = SpanGuard::span( TraceCategory::Transactions, txq_span::prefix::txq, txq_span::op::applyDirect); auto const account = (*tx)[sfAccount];