[FFmpeg-devel] R: [PATCH] MXF format fix for Sony Station compatibility

Development at axeltechnology.com Development at axeltechnology.com
Tue Oct 24 13:35:26 EEST 2017


>The patch contains trailing whitespace and tabs, both cannot be committed to the FFmpeg repository.
>You can use tools/patcheck to find the issues.

Cleaned up the spaces/tabs and added braces around else statemens.


> +    mxf_write_local_tag(pb, 4, 0x3308);
> +    if (st->codec->pix_fmt == AV_PIX_FMT_YUV420P)
> +        avio_wb32(pb, 2);
> +    else
> +        avio_wb32(pb, 1);

>This looks a little suspicious:
>What does "2" mean, what does "1" mean?

Added v_chroma_sub_sample variable instead of '1' and '2' as you suggested.


>Is there a reason why this patch does not name an author?

We are a team of developers and worked together on this patch


>Thank you, Carl Eugen

Thank you for your time
Axel Technology development team


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Sony-XDCAM-Fix.patch
Type: application/octet-stream
Size: 18427 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171024/712cbd46/attachment.obj>


More information about the ffmpeg-devel mailing list