From: Mathieu Duponchelle <mathieu@centricular.com>
Date: Mon, 30 Sep 2019 14:11:18 +0000 (+0200)
Subject: device-monitor: fix device listing without --follow
X-Git-Tag: 1.19.3~511^2~919
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0e1da383a7bb9083184779cced152c0a57442b14;p=platform%2Fupstream%2Fgstreamer.git

device-monitor: fix device listing without --follow

In !427, I removed the call to get_devices in order to always
print added devices from the bus handler, however this requires
the main loop to run until all pending messages have been consumed.

This commit achieves this by always running the main loop, and
simply adding an idle source to quit it in the non --follow case.
---

diff --git a/tools/gst-device-monitor.c b/tools/gst-device-monitor.c
index 5765de4..2b802c3 100644
--- a/tools/gst-device-monitor.c
+++ b/tools/gst-device-monitor.c
@@ -268,6 +268,13 @@ bus_msg_handler (GstBus * bus, GstMessage * msg, gpointer user_data)
   return TRUE;
 }
 
+static gboolean
+quit_loop (GMainLoop * loop)
+{
+  g_main_loop_quit (loop);
+  return G_SOURCE_REMOVE;
+}
+
 int
 main (int argc, char **argv)
 {
@@ -364,12 +371,16 @@ main (int argc, char **argv)
 
   GST_INFO ("Took %.2f seconds", g_timer_elapsed (timer, NULL));
 
-  if (follow) {
+  if (!follow) {
+    /* Consume all the messages pending on the bus and exit */
+    g_idle_add ((GSourceFunc) quit_loop, app.loop);
+  } else {
     g_print ("Monitoring devices, waiting for devices to be removed or "
         "new devices to be added...\n");
-    g_main_loop_run (app.loop);
   }
 
+  g_main_loop_run (app.loop);
+
   gst_device_monitor_stop (app.monitor);
   gst_object_unref (app.monitor);