feat: add new collection type, BoundedVec, to the bounded collections crate#2199
feat: add new collection type, BoundedVec, to the bounded collections crate#2199
BoundedVec, to the bounded collections crate#2199Conversation
…sharifi/add-bounded-vec
BoundedVec, to the bounded collectiosn crateBoundedVec, to the bounded collections crate
Code Review:
|
There was a problem hiding this comment.
Most of the implementation is taken from https://github.com/ergoplatform/bounded-vec/blob/develop/src/bounded_vec.rs (license allows this).
Some small changes have been made, such as not implementing AsMut since that can break the invariants.
Borsh and scehma implementations have also been added for ABI generation.
gilcu3
left a comment
There was a problem hiding this comment.
Thank you!
Left a few questions, mostly related to design choices
part of #2186
This PR is broken out of #2165, by only creating the bounded vec type, and not integrating it into the contract yet. See slack thread for motivation https://nearone.slack.com/archives/C0912BTG51T/p1771583540732579?thread_ts=1771583231.086569&cid=C0912BTG51T