On Sun, 10 May 2015, Martin Storsjö wrote:

Hi,

On Sun, 10 May 2015, James Almer wrote:

id should be an integer, not a string. It is also optional, so use
contentType instead which is the proper attribute for these values.

I don't see anything in ISO/IEC 23009-1/2014 about id needing to be an integer - can you clarify where that requirement comes from? Otherwise we could scrap/reformulate this part of the commit message.

Oh, I found it in the "XML syntax" part in the standard, even though it wasn't mentioned in the verbal description of the id field.

The patch is ok then.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to