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

Borrowed Accounts API Rewrite #4280

Open
5 tasks
cali-jumptrading opened this issue Feb 24, 2025 · 1 comment
Open
5 tasks

Borrowed Accounts API Rewrite #4280

cali-jumptrading opened this issue Feb 24, 2025 · 1 comment
Assignees
Labels

Comments

@cali-jumptrading
Copy link
Contributor

cali-jumptrading commented Feb 24, 2025

Rewrite of borrowed accounts to make code cleaner and simpler.
Tasks:

  • Consolidate Borrowed Account API checks. All other wrapper borrowed account APIs (in instr context and txn context) should use the same borrowed account API. Borrowed Accounts API Consolidation #4306
  • Ensure that we're not missing any checks in the account access apis that Agave has
  • Simplify storage of account pubkeys and borrowed account pointers in instruction context. We should just store indices into the transaction context.
  • use function vtable to make borrowed accounts behave more like a class?
  • more stuff coming...
@cali-jumptrading cali-jumptrading self-assigned this Feb 24, 2025
@topointon-jump
Copy link
Contributor

topointon-jump commented Feb 24, 2025

  • Ensure that we're not missing any checks in the account access api's that Agave has

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants