[Bug 733070] scenario: add set_property scenario action

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Jul 11 13:00:33 PDT 2014


https://bugzilla.gnome.org/show_bug.cgi?id=733070
  GStreamer | gst-devtools | unspecified

--- Comment #4 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-07-11 20:00:28 UTC ---
(In reply to comment #3)
> Review of attachment 280528 [details]:
> 
> Just make "target-element-name" a mandatory field and push.
> 
> ::: validate/gst/validate/gst-validate-scenario.c
> @@ +868,3 @@
> +  gboolean ret;
> +
> +  name = gst_structure_get_string (action->structure, "target-element-name");
> 
> Why not make it a mandatory field then?

Forgot to ask if you see any use case for having 'target-element-type' to set
the same property to all elements of the same type.

But I guess we can always add a set_property_to_type or something like this
later if needed.

Will update and push.

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