2016-09-26 1:52 GMT+02:00 Jan Ekström <[email protected]>:
> * Renames the `width` and `height` entries to `stored_{width,height}`
> according to the specification's naming.
This should be a separate patch. (Although I wonder how useful it is:
Are you planning to add yourself as mxf maintainer? This would be
very welcome.)
> * Switches the data type of widths and heights to uint32_t according
> to specification, adds additional checks to make sure we don't
> overflow INT_MAX as codecpar->{width,height} are signed integers.
If this fixes anything (I am not sure I understand: Are you changing a
type which introduces a possible undefined behaviour, so you add an
additional check at the same time?) it should be part of above new
patch imo (or an independent patch).
> * Adds and parses the sampled and display width and height entries.
The rest should be added together with the side data injection.
Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
