gst/gstutils.c: Add FIXME for 0.11 to set the pad as message source and not the eleme...
authorSebastian Dröge <slomo@circular-chaos.org>
Wed, 17 Dec 2008 09:37:47 +0000 (09:37 +0000)
committerSebastian Dröge <slomo@circular-chaos.org>
Wed, 17 Dec 2008 09:37:47 +0000 (09:37 +0000)
Original commit message from CVS:
* gst/gstutils.c: (gst_element_found_tags_for_pad):
Add FIXME for 0.11 to set the pad as message source and not
the element. Otherwise it's impossible to detect for which
pad the tags were found without adding an event probe
or something similar to the pad.

ChangeLog
gst/gstutils.c

index f204376..ce4cd60 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2008-12-17  Sebastian Dröge  <sebastian.droege@collabora.co.uk>
+
+       * gst/gstutils.c: (gst_element_found_tags_for_pad):
+       Add FIXME for 0.11 to set the pad as message source and not
+       the element. Otherwise it's impossible to detect for which
+       pad the tags were found without adding an event probe
+       or something similar to the pad.
+
 2008-12-16  Wim Taymans  <wim.taymans@collabora.co.uk>
 
        * docs/faq/general.xml:
index e778fb8..3442b09 100644 (file)
@@ -3230,6 +3230,9 @@ gst_element_found_tags_for_pad (GstElement * element,
   g_return_if_fail (list != NULL);
 
   gst_pad_push_event (pad, gst_event_new_tag (gst_tag_list_copy (list)));
+  /* FIXME 0.11: Set the pad as source to make it possible to detect for
+   * which pad the tags are actually found.
+   */
   gst_element_post_message (element,
       gst_message_new_tag (GST_OBJECT (element), list));
 }