MPEGGroup / FileFormat

MPEG file format discussions
24 stars 0 forks source link

'svop' vs 'svdr' #47

Closed podborski closed 4 months ago

podborski commented 2 years ago

SVCDependencyRangeBox is defined with 'svop' in the syntax but svdr is listed in definition. Which one is correct?

I assume we should keep svdr as this one is also registered at mp4ra. And change:

class SVCDependencyRangeBox extends Box('svop') ...

to

class SVCDependencyRangeBox extends Box('svdr') ...
yekuiwang commented 1 year ago

I will handle this.

podborski commented 1 year ago

@yekuiwang is this one already addressed? I don't remember if we put it in the amendment or not.

yekuiwang commented 1 year ago

@yekuiwang is this one already addressed? I don't remember if we put it in the amendment or not.

Oh I man I forgot this one. I just took a note in my to-do list to make sure to include this into the FDIS text to be output from this Oct. 2023 MPEG meeting.

yekuiwang commented 1 year ago

@podborski Aha, that was actually integrated into Amd.1! Just integrated into the FDIS text I am editing anyway.

cconcolato commented 5 months ago

@yekuiwang @podborski can one of you confirm and close this issue?

yekuiwang commented 5 months ago

Confirmed. Closing now.

yekuiwang commented 5 months ago

@cconcolato I could not figure out how to close this issue; probably I cannot. @podborski you should be able to.