[gstreamer-bugs] [Bug 396774] Make GstElementDetails extensible

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=396774
  GStreamer | gstreamer (core) | git

--- Comment #27 from Stefan Kost (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> 2010-08-13 14:40:24 UTC ---
Okay, so what about using the GstStructure for the new fields (and in 0.11 for
the existing ones too), but add _get|set functions. Thus no
gst_element_factory_get_detail() or what so ever, but:

  gst_element_class_set_doc_uri ()
  gst_element_class_set_icon_name()

  gst_element_factory_get_doc_uri ()
  gst_element_factory_get_icon_name ()

In gst-inspect we add new calls, when ever there are new fields. String based
getters will return NULL in case the detail does not exist. If we add e.g. int
based getters we need to do as in gst_structure (out variable, plus boolean
return).

> /*< private >*/
> GstStructure		 *meta_data;

I would tend to keep this as it is. But I can also make it a gpointer and cast
in the few methods using it.

I'll make a new patch if that sounds like a good plan.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.