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.

Sign up for GitHub

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

Jump to bottom

Remove unsafe in zero copy implementation #241

Open

jgur-psyops wants to merge 3 commits into main from safe-zero-copy

Contributor

jgur-psyops commented Aug 23, 2024

Changes to be addressed in api/fe:

MarginfiAccount.lending_account.balances[n].active is now a u8 (was bool). A number should cast to bool in TS without issue.
(maybe) some padding was split across multiple variables to adhere to suggested slice size limits.


          Safe zero copy implementation with no change to existing byte layout

6fbdd7d

jgur-psyops added the low priority label

jgur-psyops requested review from jkbpvsc and losman0s as code owners

August 23, 2024 07:43

jgur-psyops added 2 commits

August 23, 2024 03:48


          Fix cli

2c263bd


          fix fuzz attempt 1

283c466

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels