[Bug 733070] scenario: add set_property scenario action

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Jul 11 12:23:17 PDT 2014


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

Thibault Saunier <tsaunier> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #280528|none                        |needs-work
             status|                            |

--- Comment #3 from Thibault Saunier <tsaunier at gnome.org> 2014-07-11 19:23:14 UTC ---
Review of attachment 280528:
 --> (https://bugzilla.gnome.org/review?bug=733070&attachment=280528)

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?

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