Skip to content
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 generic assert_eq #2

Draft
wants to merge 56 commits into
base: log-type-skip-228
Choose a base branch
from
Draft

Conversation

bufferhe4d
Copy link
Owner

Using zksecurity#231

I disabled the argument type checking for assert_eq. Then implemented the recursive comparison for Array and structs. Note that we don't need to handle GenericSizedArrays since they will be monophormized by the time assert_eq is executed.

There is also a test for checking some nested structs and Arrays.

@mimoo
Copy link

mimoo commented Dec 15, 2024

why is this one a draft :o ?

@bufferhe4d
Copy link
Owner Author

why is this one a draft :o ?

I fortgot to target the main branch after zksecurity#231 was merged. Creating a full pr now.

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.

6 participants