-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support backref for wasm #395
Conversation
Pull Request Test Coverage Report for Build 8829620756Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider moving ALLOW_BACKREFS
to wasm
api, or simply exporting node_to_bytes_backrefs
separately.
Co-authored-by: Arvid Norberg <[email protected]>
I'll move |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
my main comment is that I don't think node_to_bytes_backrefs()
should have a default size limit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
Added support compressed form of clvm byte representation.
Changed
wasm/src/api.rs
towasm/src/run_program.rs
allow_backrefs()
method toFlag
structrun_clvm
requiresflag
argument. IfALLOW_BACKREFS
bit is set, it deserializes bytes input of clvm objects as compressed forms.ALLOW_BACKREFS
bit is set when callingrun_chia_program
, it deserializes bytes input of clvm objects as compressed forms.Added
sexp_from_bytes
function.to_bytes()
method toLazyNode