[FFmpeg-devel] [PATCH 2/4] aacenc: add recognized profiles array

Anssi Hannula anssi.hannula
Fri Jan 21 23:51:34 CET 2011


On 22.01.2011 00:49, Nathan Caldwell wrote:
> On Fri, Jan 21, 2011 at 1:49 PM, Anssi Hannula <anssi.hannula at iki.fi> wrote:
>> On 21.01.2011 22:31, Nathan Caldwell wrote:
>>> On Fri, Jan 21, 2011 at 12:21 PM, Anssi Hannula <anssi.hannula at iki.fi> wrote:
>>>> ---
>>>>  libavcodec/aacenc.c |    6 ++++++
>>>>  1 files changed, 6 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/libavcodec/aacenc.c b/libavcodec/aacenc.c
>>>> index c52ffa0..762e0c8 100644
>>>> --- a/libavcodec/aacenc.c
>>>> +++ b/libavcodec/aacenc.c
>>>> @@ -636,6 +636,11 @@ static av_cold int aac_encode_end(AVCodecContext *avctx)
>>>>     return 0;
>>>>  }
>>>>
>>>> +static const AVProfile profiles[] = {
>>>> +    { FF_PROFILE_AAC_LOW,  "Low"  },
>>>
>>> Couple of nits: I would prefer FF_PROFILE_AAC_LC here.
>>
>> Well, FF_PROFILE_AAC_LOW already exists, but of course "Low" can be
>> changed to "LC".
> 
> That's fine then. I had about 2 seconds before I needed to leave to
> work, and didn't get a chance to look over any more than this patch.
> 
>>> Also, is there
>>> a limit to the string length? "Low Complexity" would probably be
>>> better descriptive name.
>>
>> Well, I'd favor a shorter string as this is shown in e.g. avcodec_string().
> 
> What about "LC" then?

Would be my first choice as well, I only used "Low" due to the existing
#define. I'll change it for the resent patchset :)

-- 
Anssi Hannula



More information about the ffmpeg-devel mailing list