[Bug 743248] Add new control binding GstAppControlBinding
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Thu Jan 22 04:22:53 PST 2015
https://bugzilla.gnome.org/show_bug.cgi?id=743248
GStreamer | gstreamer (core) | unspecified
Heinrich Fink <hfink> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #295004|0 |1
is obsolete| |
--- Comment #6 from Heinrich Fink <hfink at toolsonair.com> 2015-01-22 12:22:49 UTC ---
Created an attachment (id=295163)
View: https://bugzilla.gnome.org/attachment.cgi?id=295163
Review: https://bugzilla.gnome.org/review?bug=743248&attachment=295163
updated GstAppControlBinding patch
I have addressed all the issues mentioned above, including
- Optionally use callbacks for performance reasons (see
gst_app_control_binding_set_callbacks). I also added new unit tests for this.
- Split unit tests into sync, get_value, get_value_array, get_g_value_array, as
mentioned in IRC
One thing that I wasn't quite sure about, is whether it is expected that the
binding object is unlocked (GST_OBJECT_UNLOCK) during the
callbacks/notify-destroy function or not. As it is now, the object is locked.
Please check if I should change this behavior.
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list