On 21 November 2016 at 21:47, Ronald S. Bultje <[email protected]> wrote:
> Hi, > > On Mon, Nov 21, 2016 at 4:40 PM, Rostislav Pehlivanov <[email protected] > > > wrote: > > > On 21 November 2016 at 20:44, Martin Vignali <[email protected]> > > 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 > > > [email protected] > > > 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 > [email protected] > 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 [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
