Skip to content

Fix defer instance of Speedy.Control - #668

Closed
puffnfresh wants to merge 1 commit into
digital-asset:mainfrom
puffnfresh:fix-control-stack-safety
Closed

puffnfresh wants to merge 1 commit into
digital-asset:mainfrom
puffnfresh:fix-control-stack-safety

Conversation

@puffnfresh

Copy link
Copy Markdown

This previously just eagerly evaluated the deferred computation. We just need to have a constructor which suspends the deferred computation and then unwraps it in the existing tail recursive loop.

This previously just eagerly evaluated the deferred computation. We
just need to have a constructor which suspends the deferred
computation and then unwraps it in the existing tail recursive loop.
@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

✅ All required contributors have signed the CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@puffnfresh

Copy link
Copy Markdown
Author

I have hereby read the Digital Asset CLA and agree to its terms

@remyhaemmerle-da remyhaemmerle-da left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for this, good catch.

@remyhaemmerle-da

Copy link
Copy Markdown

I cherry-picked the changes to the internal repo. It will be appeared in the external repo i few days.
Thanks for your contribution.

auto-merge was automatically disabled September 25, 2026 16:52

Pull request was closed

@github-actions github-actions Bot locked and limited conversation to collaborators Sep 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants