Skip to content

Commit

Permalink
Remove range attribute from element with a (large) restriction section
Browse files Browse the repository at this point in the history
The restriction mentions the value that are allowed. The range gives the lower
and upper limits. But without an indication on what to do with values in the
range but with an unknown value we have more confusion than necessary.

If the restriction values are later expanded older parser will ignore the new
values no matter what. The range is only useful to tell if the value is legit
or not, but there's still plenty of changes it is not.
  • Loading branch information
robUx4 committed May 30, 2021
1 parent 7136c75 commit 52f1435
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ebml_matroska.xml
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ though the design allows an unlimited number).</documentation>
<documentation lang="en" purpose="definition">A unique ID to identify the Track.
This **SHOULD** be kept the same when making a direct stream copy of the Track to another file.</documentation>
</element>
<element name="TrackType" path="\Segment\Tracks\TrackEntry\TrackType" id="0x83" type="uinteger" range="1-254" minOccurs="1" maxOccurs="1">
<element name="TrackType" path="\Segment\Tracks\TrackEntry\TrackType" id="0x83" type="uinteger" minOccurs="1" maxOccurs="1">
<documentation lang="en" purpose="definition">A set of track types coded on 8 bits.</documentation>
<restriction>
<enum value="1" label="video"/>
Expand Down Expand Up @@ -438,7 +438,7 @@ or interlaced, and if applicable to declare details about the interlacement.</do
<extension type="webmproject.org" webm="1"/>
<extension type="libmatroska" cppname="VideoFlagInterlaced"/>
</element>
<element name="FieldOrder" path="\Segment\Tracks\TrackEntry\Video\FieldOrder" id="0x9D" type="uinteger" minver="4" range="0-14" default="2" minOccurs="1" maxOccurs="1">
<element name="FieldOrder" path="\Segment\Tracks\TrackEntry\Video\FieldOrder" id="0x9D" type="uinteger" minver="4" default="2" minOccurs="1" maxOccurs="1">
<documentation lang="en" purpose="definition">Declare the field ordering of the video.
If FlagInterlaced is not set to 1, this Element **MUST** be ignored.</documentation>
<restriction>
Expand Down

0 comments on commit 52f1435

Please sign in to comment.