[Bug 784379] omxh264enc: add property to control the number of B-frames

[GStreamer (GNOME Bugzilla)]

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

--- Comment #8 from Guillaume Desmottes <gdesmott at gnome.org> ---
(In reply to Nicolas Dufresne (stormer) from comment #7)
> Review of attachment 354826 [details] [review]:
> 
> ::: omx/gstomxh264enc.c
> @@ -439,2 @@
>          "can't get OMX_IndexConfigVideoAVCIntraPeriod %s (0x%08x)",
>          gst_omx_error_to_string (err), err);
> 
> How is this patch related to this bug ?

While fixing this bug we introduced a second way to configure the GOP pattern,
so it's less critical if the current one is not supported.

> Other then that, would handling
> OMX_ErrorNotImplemented be a better approach ?

Good point. Doing that.

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