View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003513 | 10000-006: Mappings | Spec | public | 2016-08-19 07:12 | 2016-12-14 18:01 |
Reporter | BjarneBostrom | Assigned To | randyarmstrong | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Summary | 0003513: Clarify the place of EncodingMask when Optional Structure is part of an inheritance chain | ||||
Description | Spec Part 6 (ver 1.03) section 5.2.6 needs in my opinion a bit clarifying that where the EncodingMask should be placed if
It does state "Structures with optional fields are encoded with an encoding mask and as a sequence of fields in the order that they appear in the definition.", but e.g. "The EncodingMask shall be always first even before supertype fields" or equivalent would make it clear. | ||||
Tags | No tags attached. | ||||
Commit Version | |||||
Fix Due Date | |||||
|
Added this text to 1.04 DRAFT 09: If a Structure with optional fields is subtyped it shall not add any new optional fields. If optional fields are required then the optional fields should be nested by creating a mandatory field with a new Structure that defines the optional fields needed for the subtype. I could not think of a way to allow this without creating a mess by either creating non-intuitive dependencies on the parent type or adding complexity with separate masks for every level in the tree (which causes a lot of problems for the XML encoding). |
|
This topic may need clarification but I do not see such a general problem that we need to disallow creating subtypes of structures with optional fields. There are companion specifications that make already heavily use of this feature. Therefore we cannot restrict something that is possible and already used. |
|
Resolved interactively during Dallas F2F. EncodingMask is extended by subtypes and stated that explicitly. |
|
agreed to changes in meeting. |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-08-19 07:12 | BjarneBostrom | New Issue | |
2016-08-30 15:38 | Jim Luth | Target Version | => 1.04 |
2016-08-30 15:39 | Jim Luth | Assigned To | => randyarmstrong |
2016-08-30 15:39 | Jim Luth | Status | new => assigned |
2016-12-04 23:26 | randyarmstrong | Note Added: 0007410 | |
2016-12-04 23:26 | randyarmstrong | Status | assigned => resolved |
2016-12-04 23:26 | randyarmstrong | Resolution | open => fixed |
2016-12-10 22:19 | Matthias Damm | Note Added: 0007506 | |
2016-12-10 22:19 | Matthias Damm | Status | resolved => feedback |
2016-12-10 22:19 | Matthias Damm | Resolution | fixed => reopened |
2016-12-14 18:01 | randyarmstrong | Note Added: 0007533 | |
2016-12-14 18:01 | randyarmstrong | Status | feedback => resolved |
2016-12-14 18:01 | randyarmstrong | Resolution | reopened => fixed |
2016-12-14 18:01 | Jim Luth | Note Added: 0007534 | |
2016-12-14 18:01 | Jim Luth | Status | resolved => closed |
2016-12-14 18:01 | Jim Luth | Fixed in Version | => 1.04 |