Closed
Description
openedon Apr 5, 2021
Issue To Be Solved
The encoding of 12 types to CBOR values can be optimized.
Suggestion
Switch from CBOR map to CBOR array for:
- StorageReferenceValue
- PathValue
- CapabilityValue
- common.AddressLocation
- LinkValue
- CompositeStaticType
- InterfaceStaticType
- ConstantSizedStaticType
- ReferenceStaticType
- DictionaryStaticType
- RestrictedStaticType
- TypeValue
Definition of Done
- Optimize encoding
- Change decoding to only support this current format (i.e. remove old backwards-compatibility decoding code)
Tests - Encode new format
- Decode new format
- Decode old format, incl. round-trip, i.e. encoding should match new encoding
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment