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

Michael Niedermayer michaelni at gmx.at
Wed Jul 27 09:43:50 CEST 2011


On Wed, Jul 27, 2011 at 01:42:56AM +0200, Stefano Sabatini wrote:
> On date Monday 2011-07-25 15:57:10 +0200, Michael Niedermayer encoded:
> > On Mon, Jul 25, 2011 at 10:58:31AM +0200, Stefano Sabatini wrote:
> [...]
> > The problem is that if its made public (which the move would do)
> > then nothing can be added or removed anymore
> > i thus think the move is not a good idea
> 
> Can you elaborate on that? I don't want to make it public, being it
> private changing the struct definition shouldn't hurt too much, since
> it will only involve libavxxx > libavutil x.y.z dependencies.

libavcodec uses ff_pix_fmt_info after the patch that moves it to
libavutil.
thus if the table layout changes the same libavcodec would be
incompatible with newer libavutil


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110727/5d7a4f4c/attachment.asc>


More information about the ffmpeg-devel mailing list