[Bug 680236] QtGStreamer doesn't wrap GstDiscoverer

[GStreamer (bugzilla.gnome.org)]

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

George Kiagiadakis <kiagiadakis.george> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #15 from George Kiagiadakis <kiagiadakis.george at gmail.com> 2012-12-12 21:15:34 UTC ---
Thanks, I have commited everything in master, together with some additional
fixes that were needed.

(In reply to comment #13)
> 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.

Yes, of course, I didn't mean it depends on X11, but just that Qt (4.x at
least) doesn't come with this feature on non-X11 platforms. In any case, I've
added this documentation note myself, without mentioning X11 explicitly.

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