[FFmpeg-devel] [PATCH] Define AVMediaType in libavutil

Stefano Sabatini stefano.sabatini-lala
Sat Mar 13 18:07:55 CET 2010


On date Saturday 2010-03-13 10:22:15 -0500, Ronald S. Bultje encoded:
> Hi,
> 
> On Sat, Mar 13, 2010 at 10:10 AM, Stefano Sabatini
> <stefano.sabatini-lala at poste.it> wrote:
> > If this is OK I can replace all occurrences of CodecType with
> > corresponding AVMediaType symbols and drop the CodecType definition,
> 
> I'd do two patches:
> 1) rename CODEC_TYPE_* to AVMEDIA_TYPE_*, including all renames in all
> files and the compat macros under a version #if
> 2) move it into libavutil
> 
> You could also move CodecType, but that lacks prefix, so I see your
> logic. But doing it this way is a little weird, it's both a move +
> rename without version protection (thus API deprecation), etc.?

Done as you suggested.

Note that we don't need to #ifdef CodecType re-definitions, as the
symbol CodecType is already redefined as AVMediaType.

Regards.
-- 
FFmpeg = Freak & Free Murdering Pacific Exxagerate God
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Define-AVMediaType-enum-and-use-it-instead-of-enum-C.patch.gz
Type: application/octet-stream
Size: 68283 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100313/33df2f01/attachment.obj>



More information about the ffmpeg-devel mailing list