Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Spec-SetupPayload] TLV examples in the specification are wrong #938

Closed
vivien-apple opened this issue Jun 1, 2020 · 2 comments
Closed

[Spec-SetupPayload] TLV examples in the specification are wrong #938

vivien-apple opened this issue Jun 1, 2020 · 2 comments
Assignees
@shana-apple @vivien-apple
Labels
feature work p1 priority 1 work
Milestone
V1.0

Comments

@vivien-apple
Copy link
Contributor

Problem

The section [8.4 TLV Examples] shows a few examples for TLV encoded data.
The context specific tag values of 8.4.1.1 seems reverted since the spec.

Also the context specific tag of 8.4.2.1 is different from the value of the previous example while those are reserved tag.

It would also be nice to have an example for a uint-32 value.

Proposed solution

Fix the examples.
@vivien-apple
Copy link
Contributor Author

Should be fixed when #10 lands in the repo of the spec.

@shana-apple shana-apple added the enhancement New feature or request label Jun 4, 2020
@woody-apple woody-apple added this to the V1.0 milestone Jun 10, 2020
@woody-apple woody-apple added V0.7 feature work p1 priority 1 work and removed V1.0 enhancement New feature or request labels Oct 22, 2020
@vivien-apple
Copy link
Contributor Author

This has been merged a while back. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shana-apple shana-apple

@vivien-apple vivien-apple

Labels
feature work p1 priority 1 work
Projects
None yet
Milestone
V1.0
Development

No branches or pull requests
3 participants
@woody-apple @shana-apple @vivien-apple