[FFmpeg-devel] [PATCH] vp9: add profile names

James Almer jamrial at gmail.com
Mon May 4 23:42:32 CEST 2015


On 03/05/15 11:35 PM, James Almer wrote:
> On 03/05/15 3:55 AM, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Sun, May 3, 2015 at 2:15 AM, James Almer <jamrial at gmail.com> wrote:
>>
>>> Signed-off-by: James Almer <jamrial at gmail.com>
>>> ---
>>>  libavcodec/avcodec.h |  3 +++
>>>  libavcodec/version.h |  4 ++--
>>>  libavcodec/vp9.c     | 22 ++++++++++++++--------
>>>  3 files changed, 19 insertions(+), 10 deletions(-)
>>>
>>> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
>>> index 8c7c420..3d5e82d 100644
>>> --- a/libavcodec/avcodec.h
>>> +++ b/libavcodec/avcodec.h
>>> @@ -2912,6 +2912,9 @@ typedef struct AVCodecContext {
>>>  #define FF_PROFILE_HEVC_MAIN_STILL_PICTURE          3
>>>  #define FF_PROFILE_HEVC_REXT                        4
>>>
>>> +#define FF_PROFILE_VP9_0                            0
>>> +#define FF_PROFILE_VP9_1                            1
>>> [..]
>>
>> +static const AVProfile profiles[] = {
>>> +    { FF_PROFILE_VP9_0, "Profile 0" },
>>> +    { FF_PROFILE_VP9_1, "Profile 1" },
>>> +    { FF_PROFILE_UNKNOWN },
>>> +};
>>
>>
>> Do people agree we should use real names here, even if they're made up?
>>
>> I believe profile 0 can simply be called "main". The significant feature in
>> profile 1 is extended colorspaces (422, 440, 444), so we can probably call
>> it "extended colorspaces". Profile 2 would then be called "high bitdepth".
>> Profile 3 would then be kind of long, since it's "extended colorspaces +
>> high bitdepth", but I don't have better ideas right now...
>>
>> Other than that (so if people disagree with the proposal above, as-is),
>> patch is fine.
>>
>> Ronald
> 
> libvpx uses "Profile X" as far as i could see, so I'd rather do the same rather
> than make up names. Besides, they can be changed any time. Only the avcodec.h
> defines need to remain constant.
> 
> If nobody comments I'll apply this as-is tomorrow.
> 

Pushed.


More information about the ffmpeg-devel mailing list