[Bug 783862] omxh264enc: also use OMX_VIDEO_PARAM_AVCTYPE to set profile and level
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Fri Jun 16 14:05:22 UTC 2017
https://bugzilla.gnome.org/show_bug.cgi?id=783862
Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nicolas at ndufresne.ca
--- Comment #6 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
(In reply to Guillaume Desmottes from comment #5)
> (In reply to Nicolas Dufresne (stormer) from comment #4)
> > Review of attachment 353893 [details] [review] [review]:
> >
> > You need to initialize the structure with valid defaults (not just the
> > profile and level), otherwise none of the Android component will work. Its
> > not standard, but it would make this patch more useful.
>
> Shouldn't the get_parameter() call initialize those?
I think so, but Android component generally don't because Android generic code
will do that already. In anycase, it makes the behaviour more consistent, as
we'll rely on the same defaults everywhere instead of random defaults per
component. What I got in my dirty branch, copied from Android code:
https://paste.fedoraproject.org/paste/PwgMWes5qD1ONVMYrUJnjQ
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list