[FFmpeg-devel] [PATCH 1/2] avio: add an inline avio_eof function as a replacement for url_feof

[Ronald S. Bultje]

Hi,

On Fri, Mar 4, 2011 at 2:33 PM, Anton Khirnov <anton at khirnov.net> wrote:
> -int url_feof(AVIOContext *s);
> +static av_always_inline int avio_eof(AVIOContext *s)
> +{
> + ? ?return s ? s->eof_reached : 0;
> +}

Why do we handle the AVIOContext == NULL case? This may be considered
unrelated, but we can remove the function altogether and simply have
them access s->eof_reached directly without it.

Same for error.

Ronald