[gstreamer-bugs] [Bug 625944] New GstDiscoverer helper library

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Sep 17 11:18:33 PDT 2010


https://bugzilla.gnome.org/show_bug.cgi?id=625944
  GStreamer | gst-plugins-base | git

--- Comment #33 from Sebastian Dröge <slomo at circular-chaos.org> 2010-09-17 18:18:31 UTC ---
- use _discoverer_uri() and not _discoverer_uri_sync(), that follows glib/gio
style

- really put it in pbutils, especially now that the video dependency is gone.
all these micro libs are not going to reduce the memory footprint and
performance of gstreamer (but the same goes for all the micro plugins which
should better be grouped...)

- IMHO GstDiscovererResult should be a simple enum and not flags. error is an
unspecified error and the 4 other failures are specified errors. they feel
disjoint

- What about the compatiblity with the GstToc stuff? Edward, did you look at
that patch and think about the relationship? :) It would be bad if discoverer
caused problems for the toc stuff or the other way around and we have one of
the APIs stable already.


Other than that I think it's all ok and can be merged.

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