From 8b56749ca33ff84589540c3fd4a747d4afcd5c4a Mon Sep 17 00:00:00 2001 From: Ed Hennis Date: Wed, 20 May 2026 18:39:46 -0400 Subject: [PATCH] Apply suggestions from Copilot code review Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- include/xrpl/basics/Number.h | 2 +- src/libxrpl/protocol/Rules.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/xrpl/basics/Number.h b/include/xrpl/basics/Number.h index 3357607d60..aaecb5d5b5 100644 --- a/include/xrpl/basics/Number.h +++ b/include/xrpl/basics/Number.h @@ -65,7 +65,7 @@ isPowerOfTen(T value) * by an STAmount - IOUs, XRP, and MPTs. It has a min value of 10^18, and a max * value of 10^19-1. "LargeLegacy" is like "Large", but preserves * a rounding error when a computation results in a mantissa of - * Number::kMAX_REP that needs to be rounded up, but rounds down + * Number::kMaxRep that needs to be rounded up, but rounds down * instead. It will maintain consistent behavior until the fixCleanup3_2_0 * amendment is enabled. * diff --git a/src/libxrpl/protocol/Rules.cpp b/src/libxrpl/protocol/Rules.cpp index eb468c3bc5..7f8978778a 100644 --- a/src/libxrpl/protocol/Rules.cpp +++ b/src/libxrpl/protocol/Rules.cpp @@ -49,7 +49,7 @@ setCurrentTransactionRules(std::optional r) // mantissas even in the absence of the other two amendments. // // If any new conditions with new amendments are added, those amendments must also be added to - // createGuards. + // useRulesGuards. bool const enableCuspRoundingFix = !r || r->enabled(fixCleanup3_2_0); bool const enableVaultNumbers = enableCuspRoundingFix || (r->enabled(featureSingleAssetVault) || r->enabled(featureLendingProtocol));