Skip to content

refactor: Make transaction invariant finalize method const#7434

Open
Tapanito wants to merge 1 commit into
tapanito/invariant-improvementfrom
tapanito/const-invariants
Open

refactor: Make transaction invariant finalize method const#7434
Tapanito wants to merge 1 commit into
tapanito/invariant-improvementfrom
tapanito/const-invariants

Conversation

@Tapanito

@Tapanito Tapanito commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Makes finalizeInvariants const across all transactors; the method only reads accumulated visit state.

High Level Overview of Change

Context of Change

API Impact

  • Public API: New feature (new methods and/or new fields)
  • Public API: Breaking change (in general, breaking changes should only impact the next api_version)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
  • Peer protocol change (must be backward compatible or bump the peer protocol version)

@xrplf-ai-reviewer xrplf-ai-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues.

Review by Claude Sonnet 4.6 · Prompt: V15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant