Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Consolidate logic between std_vec and heap_based session implementations. #94

Open
davidlenfesty opened this issue Dec 11, 2021 · 1 comment

Comments

@davidlenfesty
Copy link
Collaborator

Specifically the update function, how it's implemented in HeapSessionManager is much more readable and clear, and we can likely consolidate the logic into a single place.

At the minimum update StdVecSessionManager to mimic the style.

@teamplayer3
Copy link
Contributor

I thought on this as well. I think we can go with HeapSessionManager. This has the same logic with some improvements. It uses the alloc crate, so it can be used in no_std and std environments.

I think a more contrast implementation would be having more static memory usage.

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

No branches or pull requests

2 participants