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

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #18 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-08-06 12:00:20 UTC ---
>  (...) but I'd like to avoid having a gazillion methods.  How about :
> ....
> gboolean
> gst_discoverer_information_parse(GstDiscoverStreamInfo *i,
> GstDiscovererStreamInfo **prev, GstDiscovererStreamInfo **next, GstCaps **caps,
> GstTagList **tags, GstStructure **misc);
> 
> And then _parse for all the subclasses (having them parse the common info might
> be a good idea too for easy bindings, but then they're going to have a huge
> number of arguments).

I find this rather ugly tbh. I still prefer the 'gazillion' methods approach,
at least for the audio/video/container stream structures, because that's just
nicer and more straight-forward API IMHO, and also easier to extend when there
will be more things to extract in future (ie. just add a new getter vs. a
parse_more()).

People often have problems with out arguments (see parsing functions in core),
and this is supposed to be *nice* end-user API, the bar should be low here.

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