[FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: fix aspect ratio when writing 16:9 DV frames

Tobias Rapp t.rapp at noa-archive.com
Mon Sep 18 10:33:46 EEST 2017


On 15.09.2017 20:44, Michael Niedermayer wrote:
> On Thu, Sep 14, 2017 at 03:44:41PM +0200, Tobias Rapp wrote:
>> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
>> ---
>>   libavformat/mxfenc.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
>> index 7289e0b..da4d7b4 100644
>> --- a/libavformat/mxfenc.c
>> +++ b/libavformat/mxfenc.c
>> @@ -1810,7 +1810,7 @@ static int mxf_parse_dv_frame(AVFormatContext *s, AVStream *st, AVPacket *pkt)
>>       stype    = vs_pack[3] & 0x1f;
>>       pal      = (vs_pack[3] >> 5) & 0x1;
>>   
>> -    if ((vs_pack[2] & 0x07) == 0x02)
>> +    if ((vsc_pack[2] & 0x07) == 0x02)
>>           sc->aspect_ratio = (AVRational){ 16, 9 };
>>       else
>>           sc->aspect_ratio = (AVRational){ 4, 3 };
> 
> LGTM
> 
> thanks

Applied, thanks for the review.

Tobias



More information about the ffmpeg-devel mailing list