[FFmpeg-devel] [PATCH] gifdec: drop pointless casting

Piotr Bandurski ami_stuff at o2.pl
Thu Dec 20 20:16:20 CET 2012




Dnia 20 grudnia 2012 20:13 Paul B Mahol <onemda at gmail.com> napisał(a):

> On 12/20/12, Piotr Bandurski <ami_stuff at o2.pl> wrote:
> > Hi,
> >
> >> - if( (unsigned)s->screen_width > 32767
> >> - || (unsigned)s->screen_height > 32767){
> >> + if (s->screen_width > 32767 ||
> >> + s->screen_height > 32767) {
> >
> > Would removing this check be enough to support gifs up to 65535x65535?
> 
> There are such files?

I don't know, but specs allows such huge resolutions, so it would be good to support them:

http://www.aivosto.com/vbtips/imageformats.html

Regards



More information about the ffmpeg-devel mailing list