Skip to content

fix: debug assertions that failed by withdraw-limit OnTransferHook#1400

Merged
enthusiastmartin merged 1 commit intomasterfrom
f3/fix-debug-asserts
Mar 25, 2026
Merged

fix: debug assertions that failed by withdraw-limit OnTransferHook#1400
enthusiastmartin merged 1 commit intomasterfrom
f3/fix-debug-asserts

Conversation

@F3Joule
Copy link
Copy Markdown
Contributor

@F3Joule F3Joule commented Mar 25, 2026

Description

OnTransferHook eagerly called convert_to_hdx on every transfer, before checking whether the result is actually needed, which caused some debug assertions to fail.

How Has This Been Tested?

make test

@F3Joule F3Joule self-assigned this Mar 25, 2026
@github-actions
Copy link
Copy Markdown

Crate versions that have not been updated:

  • hydradx-runtime: v405.0.0

Runtime version has not been increased.

@F3Joule F3Joule requested review from cl0w5 and removed request for enthusiastmartin March 25, 2026 09:51
@github-actions
Copy link
Copy Markdown

Quick benchmark at commit 7c60c03 has been executed successfully.
View results

@enthusiastmartin enthusiastmartin merged commit a3054ab into master Mar 25, 2026
11 of 14 checks passed
@enthusiastmartin enthusiastmartin deleted the f3/fix-debug-asserts branch March 25, 2026 13:07
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.

3 participants