[FFmpeg-devel] [PATCH] pixcolorinfo: remove unused PixFmtInfo.depth field

Stefano Sabatini stefano.sabatini-lala at poste.it
Mon Jul 25 10:58:31 CEST 2011


On date Monday 2011-07-25 02:14:53 +0200, Michael Niedermayer encoded:
> On Sun, Jul 24, 2011 at 03:49:41PM +0200, Stefano Sabatini wrote:
> > Use information stored in pixdesc instead.
> > ---
> >  libavutil/pixcolorinfo.c |   54 ----------------------------------------------
> >  libavutil/pixcolorinfo.h |    1 -
> >  2 files changed, 0 insertions(+), 55 deletions(-)
> 
> we cannot do this if this is moved to libavutil and used from outside
> it would break ABI and would need major bumps, thats
> also why i think the move of this to lavu is not a good idea

The table was static, so was only used in imgconvert.o.

The move adds a dependency of the kind
lavc 53.A.B requires lavu >= 51.X.Y

I'll bump both lavu and lavc micro when moving the table, as done
previously in other cases (e.g. when using internally a new function
defined in lavu).

But I see that it is better to remove the field *before* to do the
move.
-- 
FFmpeg = Frightening and Fundamentalist Merciful Peaceful Enchanting Gadget


More information about the ffmpeg-devel mailing list