This PR changes OrderedMap.Get() to return Value instead of Storable.
Currently, OrderedMap.Get() returns Storable and client converts returned Storable to Value. However, it only makes sense to return Storable if client needs to remove register (slab) by StorageID (StorageIDStorable).
Get() should only provide value without possibility of client manipulating the underlying storable.
This is prep work for Atree Register Inlining (#292) and will also harden the API.
Corresponding PR for Array.Get() is PR #316.
[ ] Targeted PR against main branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
Closes #317 Updates #296 #292
Description
This PR changes
OrderedMap.Get()to returnValueinstead ofStorable.Currently,
OrderedMap.Get()returnsStorableand client converts returnedStorabletoValue. However, it only makes sense to returnStorableif client needs to remove register (slab) byStorageID(StorageIDStorable).Get()should only provide value without possibility of client manipulating the underlying storable.This is prep work for Atree Register Inlining (#292) and will also harden the API.
Corresponding PR for
Array.Get()is PR #316.mainbranchFiles changedin the Github PR explorer