[FFmpeg-devel] [PATCH] make fifo allocate AVFifoBuffer

[Reimar Döffinger]

On Sun, Mar 08, 2009 at 03:13:53PM +0100, Michael Niedermayer wrote:
> On Sun, Mar 08, 2009 at 09:25:50AM +0100, Reimar D?ffinger wrote:
> > On Sun, Mar 08, 2009 at 08:43:18AM +0100, Reimar D?ffinger wrote:
> > > On Sun, Mar 08, 2009 at 02:41:05AM +0100, Michael Niedermayer wrote:
> > > > Patch below should reduce future API/ABI issues with fifo.c/h changes.
> > > > 
> > > > Does anyone want me to bump major for this?
> > > 
> > > If you don't bump libavutil major, people will have a hard time keeping
> > > libavformat from the previous major version around.
> 
> i know and i will bump but iam not happy ...

Too late I guess, but (if you ignore fifo.h being public) it would have been enough
to decide to bump all major version together. But that gets complex
quite fast...

> > > Also avfifo_read should be removed if major version is bumped.
> > 
> > Here's a patch for that, though it may be a bit overkill if it will be
> > removed really soon.
> 
> I need one to remove av_fifo_read() and commit is fine no patch needed

Done. I will also remove the other stuff that is disabled then as soon as you
bump major (if you haven't already done so by then).