[Bug 680236] QtGStreamer doesn't wrap GstDiscoverer

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Dec 12 03:36:05 PST 2012


https://bugzilla.gnome.org/show_bug.cgi?id=680236
  GStreamer | qt-gstreamer | git

--- Comment #13 from Mathias Hasselmann (IRC: tbf) <mathias.hasselmann at gmx.de> 2012-12-12 11:36:01 UTC ---
(In reply to comment #10)

> +    /*! Allow asynchronous discovering of URIs to take place. An event loop
> must be
> +     * available for QGst::Discoverer to properly work in asynchronous mode.
> +     * \ingroup async
> +     */
> +    void start();
> 
> This needs a big fat warning that it requires a *GLIB* event loop and therefore
> it is only going to work on X11-based platforms (in Qt4 at least).
> 

I agree that GLib must be mentioned explicitly. In bold font even maybe. I
don't entirely agree with the conclusions about requiring X11: We are using
GStreamer already, so whatever imaginary non-X11 platform we deal with, you
already have a fully functional GLib event loop. Only thing that's missing is
the integration with the Qt event loop. So it should be sufficient to take
QEventDispatcherGlib and adopt it for the non-X11 platform.

http://qt.gitorious.org/qt/qt/blobs/4.8/src/corelib/kernel/qeventdispatcher_glib.cpp
http://qt.gitorious.org/qt/qt/blobs/4.8/src/corelib/kernel/kernel.pri

Actually I don't see any references to X11 in that code. So it might be
sufficient to just take this code and plug it.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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