[FFmpeg-devel] [PATCH 2/4] mxfenc: support 50 and 60 frame rates

Tomas Härdin tomas.hardin at codemill.se
Thu May 31 14:10:22 CEST 2012


On Thu, 2012-05-31 at 14:02 +0200, Matthieu Bouron wrote:
> On Thu, May 31, 2012 at 01:36:18PM +0200, Tomas Härdin wrote:
> > On Sun, 2012-05-27 at 14:21 +0200, Matthieu Bouron wrote:
> > > ---
> > > $title
> > > ---
> > >  libavformat/mxfenc.c |   14 ++++++++++++--
> > >  1 file changed, 12 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> > > index b2ef1f3..1d11bae 100644
> > > --- a/libavformat/mxfenc.c
> > > +++ b/libavformat/mxfenc.c
> > > @@ -44,8 +44,10 @@
> > >  #include "internal.h"
> > >  #include "mxf.h"
> > >  
> > > -static const int NTSC_samples_per_frame[] = { 1602, 1601, 1602, 1601, 1602, 0 };
> > > -static const int PAL_samples_per_frame[]  = { 1920, 0 };
> > > +static const int NTSC_samples_per_frame[]    = { 1602, 1601, 1602, 1601, 1602, 0 };
> > > +static const int NTSC_60_samples_per_frame[] = { 801, 801, 801, 801, 0 };
> > 
> > Shouldn't this be something like "{ 801, 801, 801, 801, 800, 0 };"?
> > 48000/60000*1001 = 800.8 after all..
> > The rest looks fine.
> > 
> 
> Indeed ! Thanks for pointing that out.
> Patch updated

Looks OK to me.



More information about the ffmpeg-devel mailing list