tools: Use l_main_run_with_signal instead of open coding it
authorMarcel Holtmann <marcel@holtmann.org>
Mon, 3 Dec 2018 18:48:08 +0000 (19:48 +0100)
committerhimanshu <h.himanshu@samsung.com>
Tue, 21 Jan 2020 12:11:49 +0000 (17:41 +0530)
Change-Id: Idca5ab5133fad95a9480c0216ad377c777d96f61
Signed-off-by: himanshu <h.himanshu@samsung.com>
tools/btpclient.c

index 6653df2..23a0a29 100644 (file)
@@ -29,6 +29,7 @@
 #include <stdlib.h>
 #include <assert.h>
 #include <getopt.h>
+#include <signal.h>
 
 #include <ell/ell.h>
 
@@ -2833,8 +2834,7 @@ static void register_core_service(void)
                                        btp_core_unregister, NULL, NULL);
 }
 
-static void signal_handler(struct l_signal *signal, uint32_t signo,
-                                                       void *user_data)
+static void signal_handler(uint32_t signo, void *user_data)
 {
        switch (signo) {
        case SIGINT:
@@ -3162,8 +3162,6 @@ static const struct option options[] = {
 int main(int argc, char *argv[])
 {
        struct l_dbus_client *client;
-       struct l_signal *signal;
-       sigset_t mask;
        int opt;
 
        l_log_set_stderr();
@@ -3200,11 +3198,6 @@ int main(int argc, char *argv[])
 
        adapters = l_queue_new();
 
-       sigemptyset(&mask);
-       sigaddset(&mask, SIGINT);
-       sigaddset(&mask, SIGTERM);
-       signal = l_signal_create(&mask, signal_handler, NULL, NULL);
-
        dbus = l_dbus_new_default(L_DBUS_SYSTEM_BUS);
        l_dbus_set_ready_handler(dbus, ready_callback, NULL, NULL);
        client = l_dbus_client_new(dbus, "org.bluez", "/org/bluez");
@@ -3218,11 +3211,10 @@ int main(int argc, char *argv[])
 
        l_dbus_client_set_ready_handler(client, client_ready, NULL, NULL);
 
-       l_main_run();
+       l_main_run_with_signal(signal_handler, NULL);
 
        l_dbus_client_destroy(client);
        l_dbus_destroy(dbus);
-       l_signal_remove(signal);
        btp_cleanup(btp);
 
        l_queue_destroy(adapters, (l_queue_destroy_func_t)btp_adapter_free);