On 21 November 2016 at 20:44, Martin Vignali <martin.vign...@gmail.com>
wrote:

> Hello,
>
> New patchs in attach.
> Correction have been made followings comments from Andreas and Carl.
>
> @Rotislav : thanks for your answer.
>
> Martin
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
I don't think you need the checks, it was pointed out that
ff_set_dimensions already does that.
Also IIRC lavc doesn't support widths or heights over 16384 anyway.

>unsigned int pixel_size
Please use uint32_t instead of unsigned ints throughout the patches.. I'm
paranoid about unspecified sizes.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to