* XRPLF/ximinez/number-fix-maxrepcusp:
Fix more AMM tests, and to not exclude fixCleanup3_2_0
docs: Add --parallel flag to cmake build commands in BUILD.md (7302)
fix: Fix wrong hybrid offer orderbook placement and update `LedgerStateFix` to amend `ExchangeRate` meta (7087)
Change the priority of the amendments for large mantissas
Apply suggestions from @Tapanito code review
Apply suggestions from Copilot code review
Review feedback from @tapanito: lambda checks condition in doRoundUp
style: More clang-tidy identifier renaming (7290)
fix: Update pDEX invariant firing under a valid offer deletion (7118)
fix: Fix multisign and signfor to check for delegate (7064)
refactor: Fix `sfGeneric` and `sfInvalid` field names (7300)
docs: Fix some comments to improve readability (7122)
feat: Propagate underlying MPT flags to vault shares (7077)
- Order the checks so that large mantissa is only enabled if SAV or LP
are enabled. fixCleanup3_2_0 only enables the rounding fix.
- Fix tests, and don't exclude fixCleanup3_2_0 in AMM tests
- Also fix formatting
* XRPLF/ximinez/number-fix-maxrepcusp:
fix: Disable unnecessary sanity-check in VaultDeposit (7288)
ci: [DEPENDABOT] bump actions/upload-artifact from 7.0.0 to 7.0.1 (7286)
ci: Only run reusable package in public repos (7293)
fix: Set default peering port to 2459 (6848)
fix: Use account ledger entry when canceling token escrows (6171)
refactor: Rename `account_` to `accountID_` (7284)
ci: Add Linux package builds (DEB + RPM) to CI (6639)
refactor: Clean up comments post-clang-tidy changes (7283)
release: Set version to 3.3.0-b0 (7280)
refactor: Rename static constants (7120)
refactor: Use `isFlag` where possible instead of bitwise math (7278)
ci: Update XRPLF/actions (7281)
clang-tidy: this is ridiculous
clang-tidy: {}
* XRPLF/develop:
release: Set version to 3.3.0-b0 (7280)
refactor: Rename static constants (7120)
refactor: Use `isFlag` where possible instead of bitwise math (7278)
ci: Update XRPLF/actions (7281)
- Some EscrowToken tests used a hard-coded list of amendments to
determine whether to expect large mantissa logic. That ignored the
effects of fixCleanup3_2_0, especially as applied to the previous fix
affecting preflight, preclaim, etc.
- Add a helper function, useRulesGuards, which will return the same
decision as createGuards and setCurrentRulesImpl. Use that helper
function in the relevant tests.
- Also remove an #include that clang-tidy was complaining about.
- Refactor the Guard decision in withTxnType into createGuards, which
lives in Rules.cpp. It is physically located near
setCurrentTransactionRules, and documented to explain that changes
need to be synchronized.
* mywork/ximinez/number-fix-maxrepcusp:
Address more nitpicky AI comments
What is it going to take to get clang-tidy to shut up?
More clang-tidy changes: AMMExtended_test member and Number_test includes
Fix clang-tidy issues, and more AI complaints
Fix AI-identified mistakes
fix: Fix a rounding error at the Number::maxRep cusp
- Add helper function, doDropDigit, to wrap the common pattern:
push(mantissa % 10);
mantissa /= 10;
++exponent;
- Might have been helpful to catch this issue when developing.
