Skip to content
This repository has been archived by the owner on Jun 19, 2021. It is now read-only.

Replace some uses of Vec with SmallVec #45

Open
LukasKalbertodt opened this issue Jul 22, 2016 · 1 comment
Open

Replace some uses of Vec with SmallVec #45

LukasKalbertodt opened this issue Jul 22, 2016 · 1 comment
Labels
D-medium P-low T-improvement

Comments

@LukasKalbertodt
Copy link
Member

Currently we use Vec for everything, including use cases in which we expect only a few items. This is pretty inefficient as Vec always heap allocates.

At some point we should fix this...
@jonas-schievink
Copy link
Contributor

Also consider using Vec::with_capacity and replacing unnecessary Vecs with iterators.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
D-medium P-low T-improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jonas-schievink @LukasKalbertodt