[FFmpeg-devel] [libav-devel] [PATCH] fifo: add av_fifo_peek2()

Stefano Sabatini stefano.sabatini-lala at poste.it
Sat Jul 16 01:33:12 CEST 2011


On date Friday 2011-07-15 16:32:52 +0200, Michael Niedermayer encoded:
> On Thu, Jul 14, 2011 at 12:18:49AM +0200, Stefano Sabatini wrote:
> > On date Tuesday 2011-07-05 01:54:51 +0200, Stefano Sabatini encoded:
> > > On date Monday 2011-07-04 16:42:04 +0200, Diego Biurrun encoded:
> > [...]
> > > From 2ec24f2f0c79414edfd558f3c55b7d7969018912 Mon Sep 17 00:00:00 2001
> > > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > > Date: Wed, 29 Jun 2011 17:30:23 +0200
> > > Subject: [PATCH] fifo: add av_fifo_peek2()
> > > 
> > > The new function provides a more generic interface than av_fifo_peek()
> > > for peeking at a FIFO buffer data.
> > > ---
> > >  libavutil/fifo.h |   21 +++++++++++++++++++++
> > >  1 files changed, 21 insertions(+), 0 deletions(-)
> > 
> > Ping.
> 
> I still think the % are a bad idea, they probably make the code 10
> times slower ...
> and in what use case would they be needed ?

Possibly none, I thought it could be useful to return a pointer to the
buffer rather than NULL, without considering performance issues. I'll
update the patch accordingly.


More information about the ffmpeg-devel mailing list