[Bug 764316] vaapidecode: simplify downstream negotiation

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Mar 30 09:04:25 UTC 2016


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

--- Comment #4 from sreerenj <bsreerenj at gmail.com> ---
Hm, these patches are causing more regressions . Not only the vp9 sample i have
shared before.

Patch1 is trivial , just removing the function declaration, which is okay to
push 
if you want.
But others  require careful review and regression testing. So please don't push
those now.
we can think of removing more global state variables, but on the other hand I
have a feeling that it is more readable when keeping separate VideoInfo for
decoded_surface and display_info. Better optimizations also possible by
removing Caps variables from GstVaapiDecode structure and utilize the
GstVideoInfo instead.

-- 
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