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.

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

Issue: Short vec serialization is not implemented #281

Closed
Virus-Axel opened this issue Sep 15, 2024 · 0 comments · Fixed by #282
Closed

Issue: Short vec serialization is not implemented #281

Virus-Axel opened this issue Sep 15, 2024 · 0 comments · Fixed by #282
Labels
bug Something isn't working

Comments

@Virus-Axel
Copy link
Owner

Description

A byte-long length is used when short_vec serialization is desired in several places. This causes some limitations with size and problems when these limits are exceeded.

Suggested Improvements

Implement short_vec and use where faulty serialization is use. Would be great to add a test with large instruction data as well.

Extra Information

No response
@Virus-Axel Virus-Axel added the bug Something isn't working label Sep 15, 2024
@Virus-Axel Virus-Axel self-assigned this Sep 15, 2024
@Virus-Axel Virus-Axel linked a pull request Sep 29, 2024 that will close this issue
memcmp filters and filters for anchor types #282
Merged
1 task
@Virus-Axel Virus-Axel removed their assignment Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

memcmp filters and filters for anchor types barelooksrare/godot-solana-sdk
1 participant
@Virus-Axel