qa-monitor: remove reference to the runner
authorThiago Santos <thiago.sousa.santos@collabora.com>
Wed, 7 Aug 2013 19:22:36 +0000 (16:22 -0300)
committerThiago Santos <thiago.sousa.santos@collabora.com>
Wed, 7 Aug 2013 19:23:25 +0000 (16:23 -0300)
qa-monitor implements qa-reporter, and we already have a runner stored
there.

validate/gst/qa/gst-qa-monitor.c
validate/gst/qa/gst-qa-monitor.h
validate/gst/qa/gst-qa-reporter.h

index 87b14f7..f7729de 100644 (file)
@@ -20,7 +20,6 @@
  */
 
 #include "gst-qa-monitor.h"
-#include "gst-qa-reporter.h"
 
 /**
  * SECTION:gst-qa-monitor
@@ -253,10 +252,8 @@ gst_qa_monitor_set_property (GObject * object, guint prop_id,
             (GST_OBJECT_NAME (monitor->target)));
       break;
     case PROP_RUNNER:
-      /* 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);
+      gst_qa_reporter_set_runner (GST_QA_REPORTER (monitor),
+          g_value_get_object (value));
       break;
     case PROP_QA_PARENT:
       monitor->parent = g_value_get_object (value);
index ecf9052..f97bcb6 100644 (file)
@@ -25,6 +25,7 @@
 #include <glib-object.h>
 #include <gst/gst.h>
 #include "gst-qa-report.h"
+#include "gst-qa-reporter.h"
 #include "gst-qa-runner.h"
 #include "gst-qa-override.h"
 
@@ -40,7 +41,7 @@ G_BEGIN_DECLS
 #define GST_QA_MONITOR_CLASS_CAST(klass)        ((GstQaMonitorClass*)(klass))
 
 #define GST_QA_MONITOR_GET_OBJECT(m) (GST_QA_MONITOR_CAST (m)->target)
-#define GST_QA_MONITOR_GET_RUNNER(m) (GST_QA_MONITOR_CAST (m)->runner)
+#define GST_QA_MONITOR_GET_RUNNER(m) (gst_qa_reporter_get_runner (GST_QA_REPORTER_CAST (m)))
 #define GST_QA_MONITOR_GET_PARENT(m) (GST_QA_MONITOR_CAST (m)->parent)
 #define GST_QA_MONITOR_LOCK(m) (g_mutex_lock (&GST_QA_MONITOR_CAST(m)->mutex))
 #define GST_QA_MONITOR_UNLOCK(m) (g_mutex_unlock (&GST_QA_MONITOR_CAST(m)->mutex))
@@ -73,8 +74,6 @@ struct _GstQaMonitor {
 
   GstQaMonitor  *parent;
 
-  GstQaRunner   *runner;
-
   GMutex        overrides_mutex;
   GQueue        overrides;
 
index c47db41..0125c46 100644 (file)
@@ -34,6 +34,7 @@ typedef struct _GstQaReporterInterface GstQaReporterInterface;
 #define GST_QA_REPORTER(obj)                (G_TYPE_CHECK_INSTANCE_CAST ((obj), GST_TYPE_QA_REPORTER, GstQaReporter))
 #define GST_IS_QA_REPORTER(obj)             (G_TYPE_CHECK_INSTANCE_TYPE ((obj), GST_TYPE_QA_REPORTER))
 #define GST_QA_REPORTER_GET_INTERFACE(inst) (G_TYPE_INSTANCE_GET_INTERFACE ((inst), GST_TYPE_QA_REPORTER, GstQaReporterInterface))
+#define GST_QA_REPORTER_CAST(obj)           ((GstQaReporter *) obj)
 
 #ifdef G_HAVE_ISO_VARARGS
 #define GST_QA_REPORT(m, issue_id, ...)                              \