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

Audit arrayvec #46

Open
Shnatsel opened this issue Nov 1, 2019 · 1 comment
Open

Audit arrayvec #46

Shnatsel opened this issue Nov 1, 2019 · 1 comment

Comments

@Shnatsel
Copy link
Member

Shnatsel commented Nov 1, 2019 •
edited
Loading

https://crates.io/crates/arrayvec

Vec, but backed by a stack-allocated array instead of a heap allocation. Effectively 5,000 downloads/day.

(crates.io numbers are higher - 23,000 downloads/day, but 18,000 of those are due to crossbeam-epoch depending on it and I've removed the dependency in latest git).

Plenty of unsafe, and while that's more or less expected of a low-level data structure, there's still a lot of things that could go wrong (looking at you, SmallVec), so this could really use a soundness audit.
@Shnatsel
Copy link
Member Author

100% safe alternative (for some use cases) now available: https://crates.io/crates/tinyvec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Shnatsel