[FFmpeg-devel] [PATCH] avio: avio_ prefix for url_fsize

[Ronald S. Bultje]

Hi,

On Fri, Mar 4, 2011 at 1:57 PM, Anton Khirnov <anton at khirnov.net> wrote:
> ---
> ?ffmpeg.c ? ? ? ? ? ? ? ? ? | ? ?4 ++--
> ?ffplay.c ? ? ? ? ? ? ? ? ? | ? ?2 +-
> ?libavformat/apetag.c ? ? ? | ? ?2 +-
> ?libavformat/avidec.c ? ? ? | ? ?8 ++++----
> ?libavformat/avio.h ? ? ? ? | ? ?3 ++-
> ?libavformat/aviobuf.c ? ? ?| ? ?6 +++++-
> ?libavformat/cafdec.c ? ? ? | ? ?2 +-
> ?libavformat/cdg.c ? ? ? ? ?| ? ?2 +-
> ?libavformat/dv.c ? ? ? ? ? | ? ?2 +-
> ?libavformat/ffmdec.c ? ? ? | ? ?2 +-
> ?libavformat/filmstripdec.c | ? ?2 +-
> ?libavformat/flvdec.c ? ? ? | ? ?2 +-
> ?libavformat/gxfenc.c ? ? ? | ? ?2 +-
> ?libavformat/id3v1.c ? ? ? ?| ? ?2 +-
> ?libavformat/img2.c ? ? ? ? | ? ?2 +-
> ?libavformat/libnut.c ? ? ? | ? ?2 +-
> ?libavformat/mov.c ? ? ? ? ?| ? ?4 ++--
> ?libavformat/nutdec.c ? ? ? | ? ?2 +-
> ?libavformat/oggdec.c ? ? ? | ? ?2 +-
> ?libavformat/r3d.c ? ? ? ? ?| ? ?2 +-
> ?libavformat/sauce.c ? ? ? ?| ? ?2 +-
> ?libavformat/tty.c ? ? ? ? ?| ? ?2 +-
> ?libavformat/utils.c ? ? ? ?| ? ?6 +++---
> ?libavformat/vocdec.c ? ? ? | ? ?2 +-
> ?libavformat/yop.c ? ? ? ? ?| ? ?2 +-
> ?25 files changed, 37 insertions(+), 32 deletions(-)

Simple enough, looks good to me. Queued.

(Unrelated, does anyone remember why we don't use SEEK_SIZE in file.c,
which would then call stat(), to calculate filesize? The current code
- seeking to EOF and then back to the current position - seems
suboptimal to me.)

Ronald