[FFmpeg-devel] [PATCH 2/2] mxfenc: support 23.976 and 24 frame rates

[Michael Niedermayer]

On Tue, Sep 18, 2012 at 09:32:26AM +0200, Matthieu Bouron wrote:
> On Thu, Sep 13, 2012 at 12:48 AM, Matthieu Bouron <matthieu.bouron at gmail.com
> > wrote:
> 
> > ---
> >  libavformat/mxfenc.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> > index 2d6339a..46d0f04 100644
> > --- a/libavformat/mxfenc.c
> > +++ b/libavformat/mxfenc.c
> > @@ -53,6 +53,8 @@ typedef struct {
> >  } MXFSamplesPerFrame;
> >
> >  static const MXFSamplesPerFrame mxf_samples_per_frames[] = {
> > +    { { 1001, 24000 }, { 2002, 0,    0,    0,    0,    0 } }, // FILM
> > 23.976
> > +    { { 1, 24},        { 2000, 0,    0,    0,    0,    0 } }, // FILM 24
> >      { { 1001, 30000 }, { 1602, 1601, 1602, 1601, 1602, 0 } }, // NTSC
> > 29.97
> >      { { 1001, 60000 }, { 801,  801,  801,  801,  800,  0 } }, // NTSC
> > 59.94
> >      { { 1, 25 },       { 1920, 0,    0,    0,    0,    0 } }, // PAL 25
> > --
> > 1.7.12
> >
> >
> Ping :)

in the absence of other replies, this one LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120919/b998acb5/attachment.asc>