[gstreamer-bugs] [Bug 526814] [API] add gst_pad_add_{data, event, buffer}_probe_full() with disconnect notify function

[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=526814

  GStreamer | gstreamer (core) | Ver: HEAD CVS

Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #108826|0                           |1
        is obsolete|                            |
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
            Summary|Add disconnect notify to    |[API] add
                   |gst_pad_add_{data,event,buff|gst_pad_add_{data,event,buff
                   |er}_probe() functions       |er}_probe_full() with
                   |                            |disconnect notify function
   Target Milestone|HEAD                        |0.10.20




------- Comment #11 from Tim-Philipp Müller  2008-04-10 20:47 UTC -------
Thanks, committed with a few changes (most importantly: fix the argument
ordering in the non-_full functions from NULL, data to data, NULL, so that the
unit tests pass again instead of crashing):


 2008-04-10  Tim-Philipp Müller  <tim at centricular dot net>

        Based on patch by: José Alburquerque <jaalburqu at svn dot gnome dot
org>

        * gst/gstutils.c: (gst_pad_add_data_probe),
          (gst_pad_add_data_probe_full), (gst_pad_add_event_probe),
          (gst_pad_add_event_probe_full), (gst_pad_add_buffer_probe),
          (gst_pad_add_buffer_probe_full):
        * gst/gstutils.h:
        * docs/gst/gstreamer-sections.txt:
        * win32/common/libgstreamer.def:
          Add gst_pad_add_*_probe_full() functions with a notify callback that
          lets the caller free the data it passes to the probe functions. This
          is useful for bindings such as gst-python or gstreamermm (#526814).
          API: gst_pad_add_data_probe_full
          API: gst_pad_add_buffer_probe_full
          API: gst_pad_add_event_probe_full

        * tests/check/gst/gstutils.c:
          Add minimal unit test to make sure freeing the data actually works
          as expected.

        * tests/benchmarks/.cvsignore:
          Random cvsignore addendum.


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