[FFmpeg-devel] [PATCH] rename movenc.c MOVContext to MOVMuxContext

Ronald S. Bultje rsbultje
Tue Jan 13 03:03:48 CET 2009


Hi,

On Mon, Jan 12, 2009 at 8:58 PM, Baptiste Coudurier
<baptiste.coudurier at gmail.com> wrote:
> Well I agree MOVMuxContext might make more sense, however since the file
> is still called movenc.c, I feel more comfortable with MOVEncContext.

Hm...

$ grep } libavformat/*enc*.c | grep Mux
libavformat/matroskaenc.c:} MatroskaMuxContext;
libavformat/movenc.c:} MOVMuxContext;
libavformat/mpegenc.c:} MpegMuxContext;
libavformat/rmenc.c:} RMMuxContext;
$ grep } libavformat/*enc*.c | grep Enc
libavformat/vocenc.c:} VocEncContext;

No real consistency there either, I guess. Subtract RMMuxContext
because that'd be my fault (+/- 1 month ago), and MOVMuxContext is in
this patch, so basically it's 2-to-1.

Let's do a vote. :-).

Ronald




More information about the ffmpeg-devel mailing list