mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-06-06 15:38:53 +00:00
Code Issues 1 Projects Releases Wiki Activity

validate: don't store the full description struct

Browse source 
Summary:
When running valgrind we'll have 2 scenarios loaded (the normal one and
"setup_sink_props_max_lateness.scenario"). The loading code shouldn't assume
which one will contain the description it actually care about and so just look
for the fields it actually needs.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D199
This commit is contained in:
Guillaume Desmottes 2015-07-14 18:28:18 +02:00 committed by Thibault Saunier
parent 11b1ae5f3b
commit 09145cc56b
1 changed files with 17 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
validate/gst/validate/gst-validate-scenario.c
View file

@ -146,7 +146,7 @@ struct _GstValidateScenarioPrivate
GList *overrides; GList *overrides;
GstStructure *description; gchar *pipeline_name;
}; };
typedef struct KeyFileGroupName typedef struct KeyFileGroupName
@ -2197,11 +2197,17 @@ _load_scenario_file (GstValidateScenario * scenario,
type = gst_structure_get_name (structure); type = gst_structure_get_name (structure);
if (!g_strcmp0 (type, "description")) { if (!g_strcmp0 (type, "description")) {
const gchar *pipeline_name;
gst_structure_get_boolean (structure, "is-config", is_config); gst_structure_get_boolean (structure, "is-config", is_config);
gst_structure_get_boolean (structure, "handles-states", gst_structure_get_boolean (structure, "handles-states",
&priv->handles_state); &priv->handles_state);
priv->description = gst_structure_copy (structure); pipeline_name = gst_structure_get_string (structure, "pipeline-name");
if (pipeline_name) {
g_free (priv->pipeline_name);
priv->pipeline_name = g_strdup (pipeline_name);
}
continue; continue;
} else if (!(action_type = _find_action_type (type))) { } else if (!(action_type = _find_action_type (type))) {
@ -2499,8 +2505,7 @@ gst_validate_scenario_finalize (GObject * object)
(GDestroyNotify) gst_mini_object_unref); (GDestroyNotify) gst_mini_object_unref);
g_list_free_full (priv->on_addition_actions, g_list_free_full (priv->on_addition_actions,
(GDestroyNotify) gst_mini_object_unref); (GDestroyNotify) gst_mini_object_unref);
if (priv->description) g_free (priv->pipeline_name);
gst_structure_free (priv->description);
g_mutex_clear (&priv->lock); g_mutex_clear (&priv->lock);
G_OBJECT_CLASS (gst_validate_scenario_parent_class)->finalize (object); G_OBJECT_CLASS (gst_validate_scenario_parent_class)->finalize (object);
@ -2633,20 +2638,16 @@ gst_validate_scenario_factory_create (GstValidateRunner *
return NULL; return NULL;
} }
if (scenario->priv->description) { if (scenario->priv->pipeline_name &&
const gchar *pipeline_name = !g_pattern_match_simple (scenario->priv->pipeline_name,
gst_structure_get_string (scenario->priv->description, GST_OBJECT_NAME (pipeline))) {
"pipeline-name"); GST_INFO ("Scenario %s only applies on pipeline %s not %s",
scenario_name, scenario->priv->pipeline_name,
GST_OBJECT_NAME (pipeline));
if (pipeline_name && !g_pattern_match_simple (pipeline_name, gst_object_unref (scenario);
GST_OBJECT_NAME (pipeline))) {
GST_INFO ("Scenario %s only applies on pipeline %s not %s",
scenario_name, pipeline_name, GST_OBJECT_NAME (pipeline));
gst_object_unref (scenario); return NULL;
return NULL;
}
} }
scenario->pipeline = pipeline; scenario->pipeline = pipeline;