From 1a7d67c4db975f62185c2c9397c8488629551bf0 Mon Sep 17 00:00:00 2001 From: Alex Kremer Date: Tue, 10 Mar 2026 09:29:08 +0000 Subject: [PATCH] chore: Enable clang-tidy `bugprone-reserved-identifier` check (#6456) --- .clang-tidy | 2 +- src/libxrpl/tx/invariants/InvariantCheck.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/.clang-tidy b/.clang-tidy index 86839f82b8..b100938182 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -34,6 +34,7 @@ Checks: "-*, bugprone-pointer-arithmetic-on-polymorphic-object, bugprone-posix-return, bugprone-redundant-branch-condition, + bugprone-reserved-identifier, bugprone-return-const-ref-from-parameter, bugprone-shared-ptr-array-mismatch, bugprone-signal-handler, @@ -94,7 +95,6 @@ Checks: "-*, # # bugprone-crtp-constructor-accessibility, # bugprone-inc-dec-in-conditions, -# bugprone-reserved-identifier, # bugprone-move-forwarding-reference, # bugprone-switch-missing-default-case, # bugprone-optional-value-conversion, diff --git a/src/libxrpl/tx/invariants/InvariantCheck.cpp b/src/libxrpl/tx/invariants/InvariantCheck.cpp index 79c593c57c..025d2d5fec 100644 --- a/src/libxrpl/tx/invariants/InvariantCheck.cpp +++ b/src/libxrpl/tx/invariants/InvariantCheck.cpp @@ -488,8 +488,9 @@ AccountRootsDeletedClean::finalize( return false; } // Simple types - for (auto const& [keyletfunc, _, __] : directAccountKeylets) + for (auto const& [keyletfunc, _1, _2] : directAccountKeylets) { + // TODO: use '_' for both unused variables above once we are in C++26 if (objectExists(std::invoke(keyletfunc, accountID)) && enforce) return false; }