[FFmpeg-devel] [PATCH] Google WebP support

[Michael Niedermayer]

On Tue, Oct 12, 2010 at 11:39:26PM +0200, Michael Niedermayer wrote:
> On Mon, Oct 11, 2010 at 11:27:08PM -0700, Pascal Massimino wrote:
> > Michael,
> > 
> > On Mon, Oct 11, 2010 at 4:59 AM, Michael Niedermayer <michaelni at gmx.at>wrote:
> > 
[...]
> > > there may be other ways to do it but this one seems simplest, other
> > > suggestions
> > > are welcome but numbered image support, per image metadata and alpha
> > > obviously
> > > should be possible in the design.
> > >
> > 
> > really, i don't see what img2.c is bringing as value. Since i still will
> > have to do the code
> > for RIFF / metadata handling (somewhere else. Possibly multiple times.). And
> > the
> 
> > use-case assumed isn't there. I don't want ./ffmpeg -i *.webp -y toto.avi to
> > work
> 
> thats ok, but i want numbered images to work. And if your implementation cannot
> handle that or more correctly noone knows how to even add it on top of that
> implementation while a img2 based one could handle it, then there is a problem
> in your implementation.

And note here iam not opposed to your implementation if you can explain how
the img2 feature set could be implemented on top of it. But i feel uneasy
without knowing if we are moving toward a dead end here



[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101012/b476c49c/attachment.pgp>