qa: Properly set reporter's runner reference

That was never set, but it is needed for the reporter to properly add
reports to the runner.
We still keep a reference on the monitor to make things simpler

diff --git a/validate/gst/qa/gst-qa-monitor.c b/validate/gst/qa/gst-qa-monitor.c
index 3259b32bc538d8d092a53d759c095e87b7a50e08..0a543610b3720a368f9af1a870a6068df5981073 100644 (file)
--- a/validate/gst/qa/gst-qa-monitor.c
+++ b/validate/gst/qa/gst-qa-monitor.c
@@ -196,6 +196,7 @@ gst_qa_monitor_set_property (GObject * object, guint prop_id,
       /* we assume the runner is valid as long as this monitor is,
        * no ref taken */
       monitor->runner = g_value_get_object (value);
+      gst_qa_reporter_set_runner (GST_QA_REPORTER (monitor), monitor->runner);
       break;
     case PROP_QA_PARENT:
       monitor->parent = g_value_get_object (value);

diff --git a/validate/gst/qa/gst-qa-reporter.c b/validate/gst/qa/gst-qa-reporter.c
index 85f4fdd8c96cf31aff22dcaafafce147c6fe4803..472e4d3bd838e4a7761abbcb10f2b33a44421843 100644 (file)
--- a/validate/gst/qa/gst-qa-reporter.c
+++ b/validate/gst/qa/gst-qa-reporter.c
@@ -155,3 +155,11 @@ gst_qa_reporter_get_runner (GstQaReporter * reporter)
 
   return priv->runner;
 }
+
+void
+gst_qa_reporter_set_runner (GstQaReporter * reporter, GstQaRunner * runner)
+{
+  GstQaReporterPrivate *priv = gst_qa_reporter_get_priv (reporter);
+
+  priv->runner = runner;
+}

diff --git a/validate/gst/qa/gst-qa-reporter.h b/validate/gst/qa/gst-qa-reporter.h
index 6991bf2ff5539304ff47334e538f697d4d09f175..b45601ca81479eeca5eac3e70f816eae4e4e127e 100644 (file)
--- a/validate/gst/qa/gst-qa-reporter.h
+++ b/validate/gst/qa/gst-qa-reporter.h
@@ -103,8 +103,8 @@ void gst_qa_report_valist                (GstQaReporter * reporter, gboolean rep
                                           GstQaReportLevel level, GstQaReportArea area,
                                           gint subarea, const gchar * format, va_list var_args);
 
+void gst_qa_reporter_set_runner          (GstQaReporter * reporter,
+                                          GstQaRunner *runner);
+
 G_END_DECLS
 #endif /* _GST_QA_REPORTER_ */
-
-
-

diff --git a/validate/gst/qa/gst-qa-scenario.c b/validate/gst/qa/gst-qa-scenario.c
index e9c5b56b98a375435319829974694bca3cbbd644..cafb47dfe3de98f14e4dc9c2f033e965ab62714d 100644 (file)
--- a/validate/gst/qa/gst-qa-scenario.c
+++ b/validate/gst/qa/gst-qa-scenario.c
@@ -374,13 +374,12 @@ static void
 gst_qa_scenario_set_property (GObject * object, guint prop_id,
     const GValue * value, GParamSpec * pspec)
 {
-  GstQaScenarioPrivate *priv = GST_QA_SCENARIO (object)->priv;
-
   switch (prop_id) {
     case PROP_RUNNER:
       /* we assume the runner is valid as long as this scenario is,
        * no ref taken */
-      priv->runner = g_value_get_object (value);
+      gst_qa_reporter_set_runner (GST_QA_REPORTER (object),
+          g_value_get_object (value));
       break;
     default:
       break;
@@ -391,13 +390,12 @@ static void
 gst_qa_scenario_get_property (GObject * object, guint prop_id,
     GValue * value, GParamSpec * pspec)
 {
-  GstQaScenarioPrivate *priv = GST_QA_SCENARIO (object)->priv;
-
   switch (prop_id) {
     case PROP_RUNNER:
       /* we assume the runner is valid as long as this scenario is,
        * no ref taken */
-      g_value_set_object (value, priv->runner);
+      g_value_set_object (value,
+          gst_qa_reporter_get_runner (GST_QA_REPORTER (object)));
       break;
     default:
       break;