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

Baptiste Coudurier baptiste.coudurier
Tue Jan 13 03:08:36 CET 2009


Ronald S. Bultje wrote:
> 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. :-).

Well, all right, let's use MOVMuxContext, so if one day the file gets 
renamed to movmux.c, it will be correctly named :>

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org




More information about the ffmpeg-devel mailing list