-
Notifications
You must be signed in to change notification settings - Fork 47
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
Move pre-defined account components' source code into masm files #960
Conversation
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.
Thank you! Looks good. I left a few comments inline.
/// This can be used in the assembly of programs that want to call procedures from this | ||
/// component. |
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.
One thing to note is that all the procedures in the basic wallet are just re-exports from miden-lib
, and so if miden-lib
is imported into an assembler, there is no need to import basic wallet code separately.
On the other hand, basic wallet code relies on miden-lib
, so - importing it without importing miden-lib
will not be sufficient.
We should probably adjust the comment to reflect the above.
This reverts commit 33f158b.
a592b03
to
005e79e
Compare
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.
All looks good! Thank you!
Closes #950
Move source code into masm files
Moves the source code of pre-defined account components into masm files in the
asm/account_components
directory. Thebuild.rs
script will build all files in that directory.