[FFmpeg-cvslog] lavf/gif: support only GIF codec.

Carl Eugen Hoyos cehoyos at ag.or.at
Thu Apr 18 15:55:50 CEST 2013


Clément Bœsch <ubitux <at> gmail.com> writes:

> On Thu, Apr 18, 2013 at 01:35:02PM +0000, Carl Eugen Hoyos wrote:
> > Clément Bœsch <git <at> videolan.org> writes:
> > 
> > > -    if (s->nb_streams != 1 || !s->streams[0]->codec ||
> > > -        s->streams[0]->codec->codec_type != AVMEDIA_TYPE_VIDEO) {
> > > +    if (s->nb_streams != 1 ||
> > > +        s->streams[0]->codec->codec_type != AVMEDIA_TYPE_VIDEO ||
> > > +        s->streams[0]->codec->codec_id   != AV_CODEC_ID_GIF) {
> > 
> > No version bump?
> 
> Why?

Didn't this commit remove a feature from libavformat?
Or do I misunderstand?

Note that I am not saying a change would make sense, 
but without any additional comment, this looks as if 
it was just missed.

> > (Did rawvideo not work anymore?)
> 
> No, rawvideo isn't supported anymore. I could make the 
> encoder write gif non-lzw packets though. Do you want this?

I have no strong opinion, sorry!

Carl Eugen



More information about the ffmpeg-cvslog mailing list