[Bug 793928] omxh265: update 422 profile names

[GStreamer (GNOME Bugzilla)]

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

Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nicolas at ndufresne.ca

--- Comment #5 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
(In reply to Julien Isorce from comment #4)
> Review of attachment 369107 [details] [review]:
> 
> ::: omx/gstomxh265utils.c
> @@ +40,2 @@
>      return (OMX_VIDEO_HEVCPROFILETYPE) OMX_ALG_VIDEO_HEVCProfileMain422;
> +  } else if (g_str_equal (profile, "main-422-10")) {
> 
> Would it be possible to re-use "gst_codec_utils_h265_get_profile" API from
> -base (and/or improve it if needed) ?

gst_codec_utils_h265_get_profile() is a parsing function (parses from
profile_tier_level data). We are converting from the caps. In general we avoid
pulling any dependency for that translation. We should fix that function to
support the new profiles (except the custom one) of course.

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