[Bug 763142] API: add async property change notification - gst_element_add_property_notify_watch()

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=763142

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #323156|none                        |reviewed
             status|                            |

--- Comment #3 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 323156:
 --> (https://bugzilla.gnome.org/review?bug=763142&attachment=323156)

Very good idea. I think keeping deep and non-deep separate is fine :)

(While we're at it, a deep-element-added/removed is something we should also
try to get in for 1.10 finally...)

::: gst/gstmessage.h
@@ +598,3 @@
+/* PROPERTY_NOTIFY */
+GstMessage *    gst_message_new_property_notify   (GstObject * src, const
gchar * property_name, GValue * val) G_GNUC_MALLOC;
+void            gst_message_parse_property_notify (GstMessage * message, const
gchar ** property_name, const GValue ** value);

Maybe add the object here too? GST_MESSAGE_SRC() basically but it would be
easier to find and often you probably want to know the sender.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.