[FFmpeg-cvslog] r22288 - trunk/libavcodec/avcodec.h

[Reimar Döffinger]

On Tue, Mar 09, 2010 at 06:29:06PM +0000, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> 
> > On Tue, Mar 09, 2010 at 11:21:46AM +0100, Michael Niedermayer wrote:
> >> 3. add a field to AVCodec that specifies the required padding for each decoder
> >
> > That's not very nice, it means and application needs a reverse data
> > path from decoder to demuxer to tell it what value it needs.
> 
> 1. open demux
> 2. open codecs
> 3. pass padding from codecs back to demux
> 4. profit
> 
> Doesn't seem that complicated to me.

Sure, as long as you don't try anything unusual.
Like try to decode with decoder A and then switch to decoder B if that failed.

> > At least it should be done in a way that an application at least
> > also has the choice of querying a padding value sufficient for all
> > codecs.
> 
> That would encourage people to use that value rather than go the
> slightly longer, but more efficient, route.

So? Not all applications have to care about 64 bytes per packet.
They could probably save more by switching their malloc implementation.
I don't see the point of a good API as finding the most complicated way of doing
something and forcing everyone to use it because 1% of the users might
need that complicated way.