[gstreamer-bugs] [Bug 491501] Reduce usage of base_init in GstElement subclasses

[GStreamer (bugzilla.gnome.org)]

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

  GStreamer | gstreamer (core) | Ver: HEAD CVS

Sebastian Dröge changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #103946|none                        |committed
               Flag|                            |
 Attachment #103949|none                        |committed
               Flag|                            |
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|0.11.0                      |0.10.18




------- Comment #28 from Sebastian Dröge  2008-02-03 10:48 UTC -------
2008-02-03  Sebastian Dröge  <slomo at circular-chaos.org>

        * gst/gstelement.c: (gst_element_base_class_init),
        (gst_element_class_add_pad_template):
        * gst/gstpadtemplate.c:
        Make it possible (and recommended) to set element details and add
        pad templates in the class_init functions by copying the details/pad
        templates in GstElement's base_init.

        Also make it possible to replace existing pad templates by adding
        a new one with the same name. This was done in a hackish fashion
        in same elements before already.

        Don't reference pad templates that are added a second time. A
        new pad template has a refcount of one and is not floating anymore
        and to be owned by the element's class. Make this more explicit by
        mentioning it in the docs of gst_element_class_add_pad_template().

        These changes are backwards compatible. Fixes bug #491501.

        * tests/check/gst/gstelement.c:
        Add unit test for setting element details, adding pad templates and
        replacing them in a subclass.


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