Mayukha Vadari
61eddd76cb
fix comments
2026-04-01 15:49:10 -04:00
Mayukha Vadari
f934239276
fix clang-tidy issues
2026-04-01 15:46:26 -04:00
Mayukha Vadari
01da501753
Merge branch 'develop' into mvadari/fix-bad-cast
2026-04-01 15:13:39 -04:00
Valentin Balaschenko
6e2452207d
fix: Remove fatal assertion on Linux thread name truncation ( #6690 )
2026-04-01 16:56:45 +00:00
Alex Kremer
29e49abd3c
chore: Enable clang-tidy coreguidelines checks ( #6698 )
...
Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com >
2026-04-01 15:46:14 +00:00
Vito Tumas
bee1056faa
fix: Enforce aggregate MaximumAmount in multi-send MPT ( #6644 )
...
Co-authored-by: xrplf-ai-reviewer[bot] <266832837+xrplf-ai-reviewer[bot]@users.noreply.github.com>
2026-04-01 13:35:13 +00:00
Mayukha Vadari
a9afd2c116
fix: Fix previous ledger size typo in RCLConsensus ( #6696 )
2026-03-31 19:56:30 +00:00
Alex Kremer
2502befb42
chore: Enable clang-tidy misc checks ( #6655 )
2026-03-31 17:29:45 +00:00
Bart
7f53351920
chore: Remove unnecessary clang-format off/on directives ( #6682 )
...
Co-authored-by: Bart <11445373+bthomee@users.noreply.github.com >
2026-03-31 15:38:04 +00:00
Pratik Mankawde
bb95a7d6cd
fix: Fix Workers::stop() race between m_allPaused and m_runningTaskCount ( #6574 )
...
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com >
2026-03-31 15:06:04 +00:00
Alex Kremer
ab8c168e3b
chore: Enable remaining clang-tidy performance checks ( #6648 )
...
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
2026-03-30 17:08:47 +00:00
Jingchen
3a477e4d01
refactor: Address PR comments after the modularisation PRs ( #6389 )
...
Signed-off-by: JCW <a1q123456@users.noreply.github.com >
Co-authored-by: Bart <bthomee@users.noreply.github.com >
2026-03-30 15:22:38 +00:00
Vito Tumas
9b944ee8c2
refactor: Split LoanInvariant into LoanBrokerInvariant and LoanInvariant ( #6674 )
2026-03-27 18:35:42 +00:00
Jingchen
addc1e8e25
refactor: Make function naming in ServiceRegistry consistent ( #6390 )
...
Signed-off-by: JCW <a1q123456@users.noreply.github.com >
Co-authored-by: Ed Hennis <ed@ripple.com >
2026-03-26 14:11:16 +00:00
Valentin Balaschenko
faf69da4b0
chore: Shorten job names to stay within Linux 15-char thread limit ( #6669 )
2026-03-26 14:10:51 +00:00
Vito Tumas
76e3b4fb0f
fix: Improve loan invariant message ( #6668 )
2026-03-26 12:40:26 +00:00
Jingchen
8f182e825a
refactor: Modularise ledger ( #6536 )
...
Signed-off-by: JCW <a1q123456@users.noreply.github.com >
Co-authored-by: Bart <bthomee@users.noreply.github.com >
Co-authored-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com >
2026-03-25 16:32:45 +00:00
Mayukha Vadari
2c7af360c2
fix: Remove unused/unreachable transactor code ( #6612 )
2026-03-25 16:02:14 +00:00
Alex Kremer
403fd7c649
fix: More clang-tidy issues found after merging to develop ( #6640 )
...
Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com >
Co-authored-by: Bart <bthomee@users.noreply.github.com >
2026-03-25 14:28:28 +00:00
Jingchen
4096623ae1
chore: Remove the forward declarations that cause build errors when unity build is enabled ( #6633 )
...
Signed-off-by: JCW <a1q123456@users.noreply.github.com >
2026-03-24 23:00:41 +00:00
Mayukha Vadari
85a4015a64
fix: Assorted Permissioned Domain fixes ( #6587 )
2026-03-24 18:53:57 +00:00
Mayukha Vadari
f7bb4018fa
fix: Assorted Vault fixes ( #6607 )
2026-03-24 18:53:49 +00:00
Alex Kremer
0eedefbf45
refactor: Enable more clang-tidy readability checks ( #6595 )
...
Co-authored-by: Sergey Kuznetsov <kuzzz99@gmail.com >
2026-03-24 15:42:12 +00:00
Mayukha Vadari
8b986e4ab0
refactor: Improve imports to only call the needed helpers ( #6624 )
2026-03-24 10:20:32 +00:00
Olek
dcfcdab14e
fix: Remove superfluous view update from credentials ( #6545 )
2026-03-23 18:29:34 +00:00
Mayukha Vadari
e0dbe90370
refactor: Move ledger entry helper functions from View.h/View.cpp to dedicated helper files ( #6453 )
...
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
2026-03-23 15:39:58 +00:00
Mayukha Vadari
be1cc48d84
fix: Assorted Oracle fixes ( #6570 )
2026-03-22 18:08:18 +00:00
Alex Kremer
12954d5392
fix: Address remaining issue after clang-tidy merge ( #6582 )
2026-03-18 22:41:09 +00:00
Jingchen
b1e5ba0518
feat: Add code generator for transactions and ledger entries ( #6443 )
...
Signed-off-by: JCW <a1q123456@users.noreply.github.com >
Co-authored-by: Bart <bthomee@users.noreply.github.com >
2026-03-18 21:11:51 +00:00
Mayukha Vadari
d360e7c5b6
refactor: Rename transactor files/classes to match the tx name ( #6580 )
2026-03-18 19:52:07 +00:00
Alex Kremer
57e4cbbcd9
refactor: Add simple clang-tidy readability checks ( #6556 )
...
This change enables the following clang-tidy checks:
- readability-avoid-nested-conditional-operator,
- readability-avoid-return-with-void-value,
- readability-braces-around-statements,
- readability-const-return-type,
- readability-container-contains,
- readability-container-size-empty,
- readability-else-after-return,
- readability-make-member-function-const,
- readability-redundant-casting,
- readability-redundant-inline-specifier,
- readability-redundant-member-init,
- readability-redundant-string-init,
- readability-reference-to-constructed-temporary,
- readability-static-definition
2026-03-18 16:41:49 +00:00
Mayukha Vadari
b92a9a3053
fix: Make assorted NFT fixes ( #6566 )
...
This change:
* Removes a set of unnecessary brackets in the initialization of an `std::uint32_t`.
* Fixes a couple of incorrect flags (same value, just wrong variables - so no amendment needed).
2026-03-18 14:47:59 +00:00
Mayukha Vadari
dcaef828b4
refactor: Replace !=/== tesSuccess with using isTesSuccess ( #6409 )
...
This change replaces all instances of `<variable> != tesSUCCESS` with `!isTesSuccess(<variable>)` and `<variable> == tesSUCCESS` with `isTesSuccess(<variable>)`.
2026-03-18 14:15:10 +00:00
yinyiqian1
6fbeb04d9e
fix: Disallow empty permission list when Delegate object is absent ( #6542 )
...
This change fixes delegation:
* If the Delegate object is not present, we should disallow empty permission list in DelegateSet preclaim.
* Empty permission list is only allowed to delete the existing Delegate object.
* In `doApply`, permission list being empty returns `tecINTERNAL`, which should not happen.
2026-03-18 13:39:31 +00:00
Alex Kremer
2a325e7e2c
chore: Enable clang-tidy bugprone-use-after-move check ( #6476 )
2026-03-18 00:12:06 +00:00
Copilot
9e14707e77
fix: Peer crawler port field type inconsistency ( #6318 )
...
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com >
Co-authored-by: mvadari <8029314+mvadari@users.noreply.github.com >
Co-authored-by: Mayukha Vadari <mvadari@gmail.com >
Co-authored-by: Mayukha Vadari <mvadari@ripple.com >
Co-authored-by: Bart <bthomee@users.noreply.github.com >
2026-03-17 22:03:56 +00:00
Mayukha Vadari
95a45d7442
chore: Add comment explaining why ammLPHolds is called twice ( #6546 )
2026-03-17 20:11:36 +00:00
tequ
b129b71c33
refactor: Use ReadView instead of ApplyView in authorizedDepositPreauth() ( #6560 )
2026-03-17 15:56:51 -04:00
tequ
013c2d6a56
refactor: Add const qualifier to SLE in verifyDepositPreauth parameter ( #6555 )
2026-03-17 19:33:18 +00:00
Alex Kremer
72f4cb097f
refactor: Enable remaining clang-tidy cppcoreguidelines checks ( #6538 )
2026-03-17 19:09:05 +00:00
Mayukha Vadari
a5185890ff
refactor: Remove dead code in escrow helper logic ( #6553 )
2026-03-17 18:13:08 +00:00
Mayukha Vadari
78b2d70a11
refactor: Assorted small DID fixes ( #6552 )
...
This change:
* Makes `addSLE` in `DIDSet` a static function, instead of a free function.
* Renames `Attestation` to `Data` everywhere (an artifact of a previous name for the field).
* Actually runs a set of tests that were not included in the `run` function of `DID_test`.
2026-03-17 14:44:07 +00:00
Mayukha Vadari
252c6768df
refactor: Clean up getFeePayer, mSourceBalance, and mPriorBalance ( #6478 )
...
This change:
* Introduces a new helper function on `STTx`, `getFeePayer`.
* Removes the usage of `mSourceBalance` and replaces it with SLE balance lookups.
* Renames `mPriorBalance` to `preFeeBalance_`
This simplifies some of the code in the transactors and makes it a lot more readable.
2026-03-17 14:12:16 +00:00
Pratik Mankawde
5ae97fa8ae
refactor: Add no-ASAN macro for Throw statements ( #6373 )
...
Throwing exceptions from code sometime confuses ASAN, as it cannot keep track of stack frames. This change therefore adds a macro to skip instrumentation around the `Throw` function.
2026-03-17 13:10:32 +00:00
Alex Kremer
7e7b71d84c
chore: Fix tests for clang-tidy bugprone-unchecked-optional-access check ( #6502 )
2026-03-16 19:47:40 -04:00
Bart
ffea3977f0
refactor: Rename system name from 'ripple' to 'xrpld' ( #6347 )
...
Per [XLS-0095](https://xls.xrpl.org/xls/XLS-0095-rename-rippled-to-xrpld.html ), we are taking steps to rename ripple(d) to xrpl(d). This change modifies the system name from `rippled` to `xrpld`.
The system name is used in limited places:
* When no explicit config file is passed via the `--config` flag, then the system name is used to construct the path where the config file and database may be stored, via the `$XDG_CONFIG_HOME` and `$XDG_DATA_HOME` directories, respectively.
* It is used in the metadata and user-agent as part of RPC calls.
* It is newly used in the full version string.
2026-03-16 21:51:31 +00:00
Alex Kremer
47a235b7be
chore: Enable clang-tidy switch-missing-default-case check ( #6461 )
2026-03-16 17:19:37 -04:00
Alex Kremer
f5e2415c98
chore: Enable clang-tidy check for CRTP constructor accessibility ( #6452 )
...
This change enables the `bugprone-crtp-constructor-accessibility` check and fixes the few compilation issues resulting from enabling it.
2026-03-16 20:18:15 +00:00
Mayukha Vadari
1a4c359351
refactor: use hasExpired in CancelCheck ( #6533 )
2026-03-16 18:14:59 +00:00
Mayukha Vadari
e4dbaf5efc
test: Remove testline JTX helper class ( #6539 )
...
This change removes the JTX helper class `testline`, which adds the line that made the `env` call, as it is no longer necessary.
2026-03-16 16:28:49 +00:00