Closed cyplo closed 5 years ago
It turns out that these are confusing and implementation is error-prone.
Changed the underlying Rust type to not allow a Vec<ObjectType> but just ObjectType and basically relied on the compiler to tell me what to fix.
Vec<ObjectType>
ObjectType
Checked that the array type cases are unaffected - see tests in tests/data/arrays.
tests/data/arrays
That's a good point :) I will add it to the cleanup PR later :)
It turns out that these are confusing and implementation is error-prone.
Changed the underlying Rust type to not allow a
Vec<ObjectType>
but justObjectType
and basically relied on the compiler to tell me what to fix.Checked that the array type cases are unaffected - see tests in
tests/data/arrays
.