[FFmpeg-devel] [PATCH 3/3] lavf/mxfenc: fix dnxhd codec uls

Tomas Härdin tomas.hardin at codemill.se
Tue Jul 10 13:34:15 CEST 2012


On Mon, 2012-07-09 at 20:01 +0200, Matthieu Bouron wrote:
> ---
> Hi,
> 
> This commit fixes the wrong codec uls i introduced with the support of the
> dnxhd codec.
> For reference: RP224 http://www.smpte-ra.org/mdd/index.html

They're not actually wrong per se - you just used version = 1 instead of
version = 10. All decent MXF readers ignore the version byte.

Anyway, the patch looks OK. Using version 10 is of course prettier,
since v10 is the latest RP224.

You could probably squash all three of these into a single patch, or at
least this and the first one since they both deal with codec UL stuff.

/Tomas



More information about the ffmpeg-devel mailing list