[FFmpeg-devel] [PATCH] avformat/mxf: fix NTSC 59.94 samples per frame layout

Marton Balint cus at passwd.hu
Sun May 27 21:15:41 EEST 2018



On Sat, 26 May 2018, Tomas Härdin wrote:

> tor 2018-05-24 klockan 22:27 +0200 skrev Marton Balint:
>> FFmbc uses this.
>> bmx uses this.
>> XAVC MXF Mapping and Operating Points prefers this.
>> Basic rounding rules also yields these numbers.
>> 
>> > Signed-off-by: Marton Balint <cus at passwd.hu>
>> ---
>>  libavformat/mxf.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/libavformat/mxf.c b/libavformat/mxf.c
>> index a909401f08..c8b57f7f0b 100644
>> --- a/libavformat/mxf.c
>> +++ b/libavformat/mxf.c
>> @@ -134,7 +134,7 @@ static const MXFSamplesPerFrame mxf_spf[] = {
>>      { { 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
>> +    { { 1001, 60000 }, { 801,  801,  800,  801,  801,  0 } }, // NTSC 59.94
>>      { { 1, 25 },       { 1920, 0,    0,    0,    0,    0 } }, // PAL 25
>>      { { 1, 50 },       { 960,  0,    0,    0,    0,    0 } }, // PAL 50
>>      { { 1, 60 },       { 800,  0,    0,    0,    0,    0 } },
>
> Looks OK

Thanks, applied.

Regards,
Marton


More information about the ffmpeg-devel mailing list