On 21 November 2016 at 21:47, Ronald S. Bultje <rsbul...@gmail.com> wrote:
> Hi, > > On Mon, Nov 21, 2016 at 4:40 PM, Rostislav Pehlivanov <atomnu...@gmail.com > > > wrote: > > > 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. > > > I don't think we typically encourage that, except for things that are > arrays... > > Ronald > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > Didn't know that, ignore it. Still though, I don't think there's a need to check the dimensions. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel