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

Make implicit prefixes standard size #75

Closed
pavel-kirienko opened this issue Mar 6, 2020 · 0 comments · Fixed by #89
Closed

Make implicit prefixes standard size #75

pavel-kirienko opened this issue Mar 6, 2020 · 0 comments · Fixed by #89
Assignees
@thirtytwobits

Comments

@pavel-kirienko
Copy link
Member

pavel-kirienko commented Mar 6, 2020 •
edited
Loading

Context: https://forum.uavcan.org/t/proposed-change-to-variable-length-arrays/740

Affected sections of the Specification:

  • 3.4.4.4 Type polymorphism and equivalency
  • 3.5.3.1 Offset attribute
  • 3.7.4.2 Variable-length array type
  • 3.7.5.1 Tagged unions

Changes: the bit length of the implicit field shall be rounded up to the smallest of 8, 16, 32, 64.

A new directive reversing this behavior may be introduced in v1.x (x>0).
pavel-kirienko added a commit to OpenCyphal/pydsdl that referenced this issue Mar 6, 2020
@pavel-kirienko
Implement OpenCyphal/specification#75
f476207
@pavel-kirienko pavel-kirienko mentioned this issue Mar 6, 2020
Merged
pavel-kirienko added a commit to OpenCyphal/public_regulated_data_types that referenced this issue Mar 6, 2020
@pavel-kirienko
Implement OpenCyphal/specification#75
03da4fe
pavel-kirienko added a commit to OpenCyphal/libcanard that referenced this issue Mar 7, 2020
@pavel-kirienko
Update the README to match OpenCyphal/specification#75
09b4370
@pavel-kirienko pavel-kirienko mentioned this issue Mar 9, 2020
Merged
@pavel-kirienko pavel-kirienko mentioned this issue Jul 12, 2020
Merged
pavel-kirienko added a commit that referenced this issue Jul 13, 2020
@pavel-kirienko
Final fixes for #75
f993558
@pavel-kirienko pavel-kirienko mentioned this issue Jul 13, 2020
Merged
pavel-kirienko added a commit that referenced this issue Jul 13, 2020
@pavel-kirienko
Merge pull request #89 from UAVCAN/truly-final-corrections-for-70
ae60db5 
Truly final corrections for #70 & #75
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thirtytwobits thirtytwobits

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Truly final corrections for #70 & #75 OpenCyphal/specification
2 participants
@pavel-kirienko @thirtytwobits