>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
0001-Sony-XDCAM-Fix.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel