2016-09-26 1:52 GMT+02:00 Jan Ekström <jee...@gmail.com>: > * 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 ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel