-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CRI vs array literal serialization #16
Comments
Small update during cleanups: The mitigation we had was tag TBD6 to distinguish dictionary references from numeric literals. That is being removed as it doesn't do anything since we switched to Packed; we may need to reintroduce something like depending on how the binary format develops. |
The current @cabo, can we put a Over all, we might consider just splitting the |
Well, the old cddl tool just crashes when you do that :-) |
We've resolved since that CRIs are typically arrays (and, once core-wg/href#62 is through, trivial to skip). Still needs text, but should be easier. |
I think that when we switched over to using CRIs everywhere (and decided that CRIs are not arrays but byte strings for CBOR sequences), we lost the ability to distinguish between CRIs and byte literals. (And similarly lost null as a possible literal value when picking the properties literal model).
Fixing this is probably not immediately practical; I think there was something in the pre-packed dictionary encoding.
We'll come back to the serialization anyway, and when we do we can have a closer look.
The text was updated successfully, but these errors were encountered: