[FFmpeg-devel] [PATCH] Introduce avio_wtag.

Ronald S. Bultje rsbultje
Fri Mar 4 12:38:45 CET 2011


Hi,

On Thu, Mar 3, 2011 at 10:28 PM, Baptiste Coudurier
<baptiste.coudurier at gmail.com> wrote:
> ---
> ?libavformat/avio.h ? ?| ? ?1 +
> ?libavformat/aviobuf.c | ? ?9 +++++++--
> ?2 files changed, 8 insertions(+), 2 deletions(-)

<nag> It's unused. </nag>

Now on a more serious note, I was OK with removing it because it's
slow, unoptimized, duplicates functionality of existing functions in a
poorly implemented way and so on.

For the readability of 99% of the cases, we added ffio_wfourcc(). For
the rest, avio_write() is sufficient because the string is always
fixed-length. What is the (real, not theoretical) case where
avio_wtag() does something that we don't already have in the
before-mentioned functions?

Besides, should probably be private, so ffio instead of avio prefix,
and I doubt that "wtag" is a good function name, but then again as
long as it's private that's not so much of an issue.

Ronald



More information about the ffmpeg-devel mailing list