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

GStreamer (bugzilla.gnome.org) bugzilla-daemon at bugzilla.gnome.org
Mon Jan 7 02:49:31 PST 2008


If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=396774

  GStreamer | gst-plugins-base | Ver: HEAD CVS




------- Comment #13 from Sebastian Dröge  2008-01-07 10:49 UTC -------
I like the idea in general but we probably should consider to fix bug #491501
(the details part) first and then get this here done.

Also, IMHO, we should deprecate all the old functions as you intended.

In gstelementfactory.c:283 you should also check for longname being correct,
and IMHO we should add a FIXME 0.11 for the description check. Nowadays it
seems to be allowed to be empty and we can't change that right now.

Then there are some changes in the registries that seem to be unrelated to this
bug, would be nice to have them as an additional patch (they seem to be fine
though).

Apart from that, I don't see anything wrong in the latest patch.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=396774.




More information about the Gstreamer-bugs mailing list