[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