-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
feat: add marshalling of structs #83
Conversation
Codecov Report
@@ Coverage Diff @@
## master #83 +/- ##
===========================================
+ Coverage 57.49% 74.42% +16.93%
===========================================
Files 119 121 +2
Lines 8375 9521 +1146
===========================================
+ Hits 4815 7086 +2271
+ Misses 3560 2435 -1125
Continue to review full report at Codecov.
|
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.
Do you mind of we hold off on this PR until I fixed the malloc linking issue correctly?
That's totally fine. The temporary fix is cumbersome in CI, so better not waste any more time on it |
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.
Very nicely done!
…structs and other types
5facf34
to
6e85fa8
Compare
Depends on #90