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

Michael Niedermayer michaelni
Tue Jan 13 21:44:38 CET 2009


On Mon, Jan 12, 2009 at 09:03:48PM -0500, 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. :-).

I vote for blue

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090113/426f7540/attachment.pgp>



More information about the ffmpeg-devel mailing list