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.

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

SampleAuxiliaryInformationOffsetsBox - allowed versions? #92

Open
bradh opened this issue Feb 9, 2024 · 1 comment
Open

SampleAuxiliaryInformationOffsetsBox - allowed versions? #92

bradh opened this issue Feb 9, 2024 · 1 comment

Comments

@bradh
Copy link

bradh commented Feb 9, 2024

The syntax for the SampleAuxiliaryInformationOffsetsBox in ISO/IEC 14496-12:2022 has options for the size of the offset array that looks like:

if ( version == 0 ) {
    unsigned int(32) offset[ entry_count ];
}
else {
    unsigned int(64) offset[ entry_count ];
}

Is the intention that the version field can be anything? Would it be (technically) valid to write a version of 0xFF and expect that any implementation could correctly parse it, at least at iso6 or higher? Or should the version always be 0x00 or 0x01?

@podborski
Copy link
Member

The version should always be 0 or 1 in this case. I raised the same issue some time ago at MPEG. I think we should fix all the syntax to something like:

if ( version == 0 ) {
    unsigned int(32) offset[ entry_count ];
}
else if (version == 1) {
    unsigned int(64) offset[ entry_count ];
}

And make sure that if an unspecified version is used there would be no syntax elements defined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants