From d40651496640012c50360bce26e7cbc399d0b634 Mon Sep 17 00:00:00 2001 From: Mike Brady Date: Tue, 24 Oct 2017 10:30:20 +0100 Subject: [PATCH] Correct a few warnings that show upo in Clang but not in GCC --- dbus/src/dbus_service.c | 1 + dbus/src/shairport-sync-dbus-client.c | 5 ++++- player.h | 1 + shairport.c | 1 + 4 files changed, 7 insertions(+), 1 deletion(-) diff --git a/dbus/src/dbus_service.c b/dbus/src/dbus_service.c index 28103243..457f3d82 100644 --- a/dbus/src/dbus_service.c +++ b/dbus/src/dbus_service.c @@ -89,4 +89,5 @@ int start_dbus_service() { g_bus_own_name(G_BUS_TYPE_SYSTEM, "org.gnome.ShairportSync", G_BUS_NAME_OWNER_FLAGS_NONE, NULL, on_name_acquired, NULL, NULL, NULL); // G_BUS_TYPE_SESSION or G_BUS_TYPE_SYSTEM + return 0; // this is just to quieten a compiler warning } diff --git a/dbus/src/shairport-sync-dbus-client.c b/dbus/src/shairport-sync-dbus-client.c index 6b576884..0a30adf6 100644 --- a/dbus/src/shairport-sync-dbus-client.c +++ b/dbus/src/shairport-sync-dbus-client.c @@ -65,9 +65,10 @@ void *dbus_thread_func(void *arg) { g_main_loop_run(loop); // dbus_service_main(); // let it run inside a thread + return NULL; // this is just to quieten a compiler warning. } -void main(void) { +int main(void) { pthread_create(&dbus_thread, NULL, &dbus_thread_func, NULL); @@ -116,4 +117,6 @@ void main(void) { printf("exiting program.\n"); g_object_unref(proxy); + + return 0; } diff --git a/player.h b/player.h index a70fa045..294c776a 100644 --- a/player.h +++ b/player.h @@ -2,6 +2,7 @@ #define _PLAYER_H #include +#include #include "config.h" diff --git a/shairport.c b/shairport.c index 7f51a358..e7c05f0f 100644 --- a/shairport.c +++ b/shairport.c @@ -969,6 +969,7 @@ pthread_t dbus_thread; void *dbus_thread_func(void *arg) { loop = g_main_loop_new(NULL, FALSE); g_main_loop_run(loop); + return NULL; } #endif