[FFmpeg-devel] [PATCH] Split H.261into separate encoder and decoder

[Diego Biurrun]

On Mon, May 21, 2007 at 02:50:53PM +0200, Michael Niedermayer wrote:
> 
> On Mon, May 21, 2007 at 02:45:47PM +0200, Aurelien Jacobs wrote:
> > 
> > Also another remarks:
> > 
> > > --- libavcodec/Makefile	(revision 9088)
> > > +++ libavcodec/Makefile	(working copy)
> > > @@ -83,8 +83,8 @@
> > >  OBJS-$(CONFIG_FRAPS_DECODER)           += fraps.o
> > >  OBJS-$(CONFIG_GIF_DECODER)             += gifdec.o lzw.o
> > >  OBJS-$(CONFIG_GIF_ENCODER)             += gif.o
> > > -OBJS-$(CONFIG_H261_DECODER)            += h261.o
> > > -OBJS-$(CONFIG_H261_ENCODER)            += h261.o
> > > +OBJS-$(CONFIG_H261_DECODER)            += h261.o h261dec.o
> > > +OBJS-$(CONFIG_H261_ENCODER)            += h261.o h261enc.o
> > 
> > I prefer seeing h261.o after h261dec.o and after h261enc.o, but
> > that's purely cosmetics and only my personnal feeling so that
> > don't matter much.
> 
> iam fine with either unless some linker on some system has problems with
> one of them ...

Some linkers do care about ordering, so putting the common code after
the rest is better.

Diego