[FFmpeg-devel] libavcodec : add psd image file decoder

[Rostislav Pehlivanov]

On 21 November 2016 at 21:47, Ronald S. Bultje <rsbultje at gmail.com> wrote:

> Hi,
>
> On Mon, Nov 21, 2016 at 4:40 PM, Rostislav Pehlivanov <atomnuker at gmail.com
> >
> wrote:
>
> > On 21 November 2016 at 20:44, Martin Vignali <martin.vignali at 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 at 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 at 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.