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