MPEGGroup / FileFormat

MPEG file format discussions
20 stars 0 forks source link

Syntax for `payt` appears invalid #95

Open bradh opened 4 months ago

bradh commented 4 months ago

In ISO/IEC 14496-12:2022 Section 9.1.5 "Statistical Information", part of the definition for the hintpayloadID is:

uint(8) count;
char rtpmap_string[count];

That does not appear to be valid SDL - there is no char data type.

Its not clear what this should be (perhaps reverse from RTP docs?), but the syntax probably should be a utfstring or utf8string or uint8 array.

(If editing this section, consider making the class names into CamelCase for consistency)

mhannuksela commented 6 days ago

Thanks. The bug report is correct and similar issues also appear in other boxes.

Ballot comment DE-054 for the 2nd DIS of the ISOBMFF 8th Edition points out that none of uint(8), uint(16), uint(32), byte and char have been specified but are used in syntax structures. MPEG will decide on the ballot comment in the upcoming meeting.

We have typically not corrected syntax element/structure names to avoid potential compatibility problems with other specifications that may refer to specific syntax element/structure names of ISOBMFF.

cconcolato commented 6 days ago

The group discussed this issue and will report back on the decision after its upcoming meeting.