Skip to content
This repository has been archived by the owner on Feb 17, 2025. It is now read-only.

Review marshalling status checks #316

Closed
vo-nil opened this issue Aug 23, 2024 · 0 comments · Fixed by #317
Closed

Review marshalling status checks #316

vo-nil opened this issue Aug 23, 2024 · 0 comments · Fixed by #317
Assignees
@vo-nil

Comments

@vo-nil
Copy link
Contributor

vo-nil commented Aug 23, 2024

We very often ignore status in marshalling.
@vo-nil vo-nil self-assigned this Aug 23, 2024
vo-nil added a commit that referenced this issue Aug 23, 2024
@vo-nil
Added macro for throw/assert of marshalling status #316
e9dff5d
@vo-nil vo-nil mentioned this issue Aug 23, 2024
Merged
vo-nil added a commit that referenced this issue Aug 23, 2024
@vo-nil
Add marshalling status checks everywhere #316
e98e9f9
vo-nil added a commit that referenced this issue Aug 26, 2024
@vo-nil
Added macro for throw/assert of marshalling status #316
46a73e5 
Add marshalling status checks everywhere #316

Reverted pedersen test, updated message #316
vo-nil added a commit that referenced this issue Aug 26, 2024
@vo-nil
Added macro for throw/assert of marshalling status #316
7a23eb3 
Add marshalling status checks everywhere #316

Reverted pedersen test, updated message #316
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@vo-nil vo-nil

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added macro for throw/assert to check marshalling status NilFoundation/crypto3
1 participant
@vo-nil