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

Tomas Härdin tomas.hardin at codemill.se
Thu May 31 13:36:18 CEST 2012


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.

/Tomas



More information about the ffmpeg-devel mailing list