[FFmpeg-devel] [PATCH] Use pixdesc for avcodec_get_pix_fmt_name()

[Michael Niedermayer]

On Sat, Nov 07, 2009 at 05:28:05PM +0100, Stefano Sabatini wrote:
> Hi,
> 
> this requires anyway the pixdesc.c compilation.
> 
> Also the names in pixdesc are defined consistently using the enum
> name, while currently pix_fmt_info uses different names.
> 
> The differences are:
> 
> pix_fmt_info -> av_pix_fmt_descriptors
> ======================================
> monow        -> monowhite
> monob        -> monoblack
> xvmcmc       -> xvmc_mpeg2_mc
> xvmcidct     -> xvmc_mpeg2_idct
> 
> I'm not very fond of arbitrary name contractions so I'd prefer to
> keep the pixdesc variants and use some backward-compatible mechanism
> and drop the old variant at the next major bump.

your patch breaks backward-compatibility

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

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- 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/20091109/db19c921/attachment.pgp>