[FFmpeg-devel] [RFC] decoder/encoder naming consistency

Kostya kostya.shishkov
Fri May 22 17:44:54 CEST 2009


On Fri, May 22, 2009 at 05:40:42PM +0200, Diego Biurrun wrote:
> On Fri, May 22, 2009 at 06:25:20PM +0300, Kostya wrote:
> > On Fri, May 22, 2009 at 04:59:54PM +0200, Diego Biurrun wrote:
> > > I propose renaming files to to adhere to the following naming
> > > convention:
> > > 
> > > foodec.c - decoder for format foo
> > > fooenc.c - encoder for format foo
> > > foo.c    - common code used by both decoder and encoder
> > > 
> > > Currently we have some cases where the encoder is in fooenc. while the
> > > decoder is in foo.c, for example zmbv.c and zmbvenc.c.
> > > 
> > > This should be made consistent.  The cases where we just have a decoder
> > > are secondary, they can remain as foo.c, but when both are available, we
> > > should try to avoid the confusion.
> > > 
> > > The same applies to demuxers and muxers.
> > > 
> > > OK to rename?
> > 
> > I would prefer clear naming scheme (which should be documented as well):
> 
> I agree that it should be documented.
> 
> > * decoder or wrapper -> ${codecname}.c
> > * pure encoder -> ${codecname}enc.c
> > * if common code is present, it should be named {codecname}.c and
> >   decoder should be renamed into ${codecname}dec.c
> > 
> > FFmpeg is decoder-oriented, after all - just compare the number of
> > decoders and encoders.
> 
> That's why I said that we can keep the names in the cases where we just
> have decoders.
> 
> > Also good luck on renaming msrle.c and msrledec.c
> 
> Hmmm?

Look at their headers (and fix a typo in msrledec.[ch])
 
> Diego



More information about the ffmpeg-devel mailing list