[Bug 740325] Add a setVideoSink() method to QGst::Ui::GraphicsVideoSurface

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri May 4 14:56:16 UTC 2018


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

George Kiagiadakis <gkiagia at tolabaki.gr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |gkiagia at tolabaki.gr
         Resolution|---                         |WONTFIX

--- Comment #1 from George Kiagiadakis <gkiagia at tolabaki.gr> ---
Thank you for that patch. However, I don't find it suitable for merging. The
reason why it is not allowed to set the element on this class is because it's
too error prone. There are several ways the internal state of the various
components can get messed up and it requires a lot of checks, which are not in
your patch. I don't think that using the internal element is too much of an
annoyance.

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