Skip to content

Commit

Permalink
EBML Schema path: add grouping EBMLAtomName
Browse files Browse the repository at this point in the history
The EBMLAtomName ABNF rule was incorrectly specified: It failed to account for the higher operator precedence of concatenation vs. alternative, as stated in RFC 5234 section 3.10: https://datatracker.ietf.org/doc/html/rfc5234#section-3.10
  • Loading branch information
matthewleon authored and robUx4 committed Nov 28, 2021
1 parent 245fc25 commit 8fd9eed
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion rfc-editor/rfc8794.xml
Original file line number Diff line number Diff line change
Expand Up @@ -949,7 +949,7 @@ EBMLElement = [IsRecursive] EBMLAtomName

PathDelimiter = "\"
IsRecursive = "+"
EBMLAtomName = ALPHA / DIGIT 0*EBMLNameChar
EBMLAtomName = (ALPHA / DIGIT) 0*EBMLNameChar
EBMLNameChar = ALPHA / DIGIT / "-" / "."

GlobalPlaceholder = "(" GlobalParentOccurrence "\)"
Expand Down
2 changes: 1 addition & 1 deletion specification.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -674,7 +674,7 @@ EBMLElement = [IsRecursive] EBMLAtomName
PathDelimiter = "\"
IsRecursive = "+"
EBMLAtomName = ALPHA / DIGIT 0*EBMLNameChar
EBMLAtomName = (ALPHA / DIGIT) 0*EBMLNameChar
EBMLNameChar = ALPHA / DIGIT / "-" / "."
GlobalPlaceholder = "(" GlobalParentOccurrence "\)"
Expand Down

0 comments on commit 8fd9eed

Please sign in to comment.