[FFmpeg-devel] [PATCH 1/3] avformat/image2: allow muxing gif files.

[Michael Niedermayer]

On Thu, Nov 07, 2013 at 04:16:08AM +0100, Michael Niedermayer wrote:
> On Wed, Nov 06, 2013 at 11:39:19PM +0100, Clément Bœsch wrote:
> > On Sat, Oct 26, 2013 at 04:13:59PM +0200, Michael Niedermayer wrote:
> > > On Sat, Oct 19, 2013 at 08:47:48PM +0200, Clément Bœsch wrote:
> > > > Fixes Ticket #2936.
> > > > ---
> > > >  libavformat/gif.c     | 85 ++++++++++++++++++++++++++++++---------------------
> > > >  libavformat/gif.h     | 26 ++++++++++++++++
> > > >  libavformat/img2enc.c | 24 +++++++++++++++
> > > >  3 files changed, 101 insertions(+), 34 deletions(-)
> > > >  create mode 100644 libavformat/gif.h
> > > 
> > > this is a bit ugly
> > > what about a way to make img2enc support arbitrary muxers?
> > > that is open muxer &encoder, do 1 image, close
> > > this shouldnt need changes in gif and might come in handy for other
> > > muxers later
> > 
> > Sounds like a better idea. Not sure if I did it as you expected, new patch
> > attached.
> 
> looks much better
> should be ok if valgrind is happy

btw, if someone wants to write a fate test ...

8...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131107/622a2c58/attachment.asc>