[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