[Ffmpeg-devel] Re: [Ffmpeg-cvslog] r7573 - trunk/libavutil/fifo.c

[Roman Shaposhnik]

On Wed, 2007-01-17 at 21:11 +0100, michael wrote:
> Modified: trunk/libavutil/fifo.c
> ==============================================================================
> --- trunk/libavutil/fifo.c	(original)
> +++ trunk/libavutil/fifo.c	Wed Jan 17 21:11:23 2007
> @@ -99,9 +99,7 @@
>              memcpy(dest, f->rptr, len);
>              dest = (uint8_t*)dest + len;
>          }
> -        f->rptr += len;
> -        if (f->rptr >= f->end)
> -            f->rptr = f->buffer;
> +        av_fifo_drain(f, len);
>          buf_size -= len;
>      }

  For things like this one, would it make sense to declare
the actual function (av_fifo_drain in this particular case)
an external inline ? Or is it too risky with gcc ? 

Thanks,
Roman.