diff --git a/src/libxrpl/telemetry/MetricsRegistry.cpp b/src/libxrpl/telemetry/MetricsRegistry.cpp index 6c16c649b1..a582ab9cb5 100644 --- a/src/libxrpl/telemetry/MetricsRegistry.cpp +++ b/src/libxrpl/telemetry/MetricsRegistry.cpp @@ -539,6 +539,10 @@ MetricsRegistry::stop() #endif // XRPL_ENABLE_TELEMETRY } +// This reads provider_ when telemetry is compiled in and touches no member +// when it is not, so clang-tidy asks for it to be static. Making it static +// would give the two builds different signatures. +// NOLINTBEGIN(readability-convert-member-functions-to-static) bool MetricsRegistry::hasPipeline() const noexcept { @@ -548,6 +552,7 @@ MetricsRegistry::hasPipeline() const noexcept return false; #endif } +// NOLINTEND(readability-convert-member-functions-to-static) // ----------------------------------------------------------------- // Synchronous instrument recording — RPC metrics diff --git a/src/xrpld/rpc/detail/PathRequest.cpp b/src/xrpld/rpc/detail/PathRequest.cpp index c891111978..888546ad0a 100644 --- a/src/xrpld/rpc/detail/PathRequest.cpp +++ b/src/xrpld/rpc/detail/PathRequest.cpp @@ -39,7 +39,7 @@ #include #include #include -#include +#include // IWYU pragma: keep #include #include #include diff --git a/src/xrpld/telemetry/AppMetricGauges.cpp b/src/xrpld/telemetry/AppMetricGauges.cpp index 2d6704d449..fd223d8d42 100644 --- a/src/xrpld/telemetry/AppMetricGauges.cpp +++ b/src/xrpld/telemetry/AppMetricGauges.cpp @@ -24,6 +24,11 @@ #include +// Both name types in the constructor signature, which is compiled in either +// way, so they belong outside the telemetry guard below. +#include +#include + #ifdef XRPL_ENABLE_TELEMETRY // The app and overlay includes below are why @@ -57,7 +62,6 @@ #include #include #include -#include #include #include #include