[FFmpeg-devel] [PATCH 3/3] lavf/mxfenc: fix dnxhd codec uls
matthieu.bouron at gmail.com
Tue Jul 10 16:43:07 CEST 2012
On Tue, Jul 10, 2012 at 01:34:15PM +0200, Tomas Härdin wrote:
> 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.
I encountered a tool which checks the version number.
> 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.
Sounds like a good idea to squash all the three commits into one single
patch. Thanks for the review.
I also noticed that patch 2/3 is wrong since it force 16:9 aspect ratio
for the dv codec and not the dnxhd codec :(
New (single) patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 5344 bytes
Desc: not available
More information about the ffmpeg-devel