[FFmpeg-devel] [PATCH] dca: export profile information on decode
Anssi Hannula
anssi.hannula
Sun Jan 2 11:49:39 CET 2011
On 01.01.2011 05:09, Michael Niedermayer wrote:
> On Fri, Dec 31, 2010 at 07:26:33AM +0200, Anssi Hannula wrote:
>> On 31.12.2010 06:21, Michael Niedermayer wrote:
>>> On Wed, Dec 29, 2010 at 07:24:09AM +0200, Anssi Hannula wrote:
>>>> ---
>>>>
>>>> While waiting for that another opinion, here's another take on the
>>>> patch, this time with a more noticiable separation of the HD parser.
>> [...]
>>>> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
>>>> index c803ec6..8833849 100644
>>>> --- a/libavcodec/avcodec.h
>>>> +++ b/libavcodec/avcodec.h
>>>> @@ -32,7 +32,7 @@
>>>> #include "libavutil/cpu.h"
>>>>
>>>> #define LIBAVCODEC_VERSION_MAJOR 52
>>>> -#define LIBAVCODEC_VERSION_MINOR 101
>>>> +#define LIBAVCODEC_VERSION_MINOR 102
>>>> #define LIBAVCODEC_VERSION_MICRO 0
>>>>
>>>> #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \
>>>> @@ -2238,6 +2238,12 @@ typedef struct AVCodecContext {
>>>> #define FF_PROFILE_AAC_SSR 2
>>>> #define FF_PROFILE_AAC_LTP 3
>>>>
>>>> +#define FF_PROFILE_DTS 20
>>>> +#define FF_PROFILE_DTS_ES 30
>>>> +#define FF_PROFILE_DTS_96_24 40
>>>> +#define FF_PROFILE_DTS_HD_HRA 50
>>>> +#define FF_PROFILE_DTS_HD_MA 60
>>>
>>> options.c update missing
>>
>> This is decoding, not encoding. What kind of options.c update is needed?
>
> I think its a good idea to add the entries to options.c when the defines are
> added to avcodec.h, otherwise it will be forgotten
OK then, I'll do that in the next version of the patch.
--
Anssi Hannula
More information about the ffmpeg-devel
mailing list