[Bug 793876] h265parse: add support for 'Format range extensions profiles'

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=793876

--- Comment #6 from Guillaume Desmottes <gdesmott at gnome.org> ---
(In reply to Tim-Philipp Müller from comment #5)
> The names seem fine to me. They look nice enough and match
> 
>  
> https://en.wikipedia.org/wiki/High_Efficiency_Video_Coding#Version_2_profiles
> 
> so what's not to like? Presumably they match with the spec too?

Yeah, I just used the names from the spec (Table A.2), put them in lowercase
and replace spaces with '-'.

> My main question is whether the enum value should be 
> GST_H265_PROFILE_FORMAT_RANGE_EXTENSION or if there should be new enum
> values for all these extended profiles?

Good point. The exact semantic of this enum isn't that clear to be either. It's
not explicitly exposed in the parser API and only used to check profile_idc so
I assumed it was meant to map the semantic of this field.

On the other hand it could make sense to move the profile detection logic from
the element to the lib and have something like this:
GstH265Profile gst_h265_profile_tier_level_get_profile (GstH265ProfileTierLevel
*ptl);

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.