Add missing function attributes suggested by clang (but not by gcc)

clang is a little more enthusiastic about suggesting these.

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

diff --git a/bus/main.c b/bus/main.c
index a325cea07d168955aa732a70fcd616aeac97f16d..5e26820ae815f77ac109e54d8147779097ed95fd 100644 (file)
--- a/bus/main.c
+++ b/bus/main.c
@@ -132,6 +132,8 @@ signal_handler (int sig)
 }
 #endif /* DBUS_UNIX */
 
+static void usage (void) _DBUS_GNUC_NORETURN;
+
 static void
 usage (void)
 {
@@ -158,6 +160,8 @@ usage (void)
   exit (1);
 }
 
+static void version (void) _DBUS_GNUC_NORETURN;
+
 static void
 version (void)
 {
@@ -169,6 +173,8 @@ version (void)
   exit (0);
 }
 
+static void introspect (void) _DBUS_GNUC_NORETURN;
+
 static void
 introspect (void)
 {

diff --git a/bus/test-launch-helper.c b/bus/test-launch-helper.c
index 5d025f5f3220d5d6a4e32a61965085b4ae10f438..5872ab2028a0a0f9b4a5824cb52931ee2da29fde 100644 (file)
--- a/bus/test-launch-helper.c
+++ b/bus/test-launch-helper.c
@@ -34,6 +34,8 @@
 #error This file is only relevant for the embedded tests on Unix
 #endif
 
+static void die (const char *failure) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *failure)
 {

diff --git a/bus/test-main.c b/bus/test-main.c
index 9496dd2a7c2741ec3cc42481fea01e89177718a5..4d3df87a4c54270a139851e099e2ffd53ce45432 100644 (file)
--- a/bus/test-main.c
+++ b/bus/test-main.c
@@ -39,6 +39,8 @@
 # include <dbus/dbus-sysdeps-unix.h>
 #endif
 
+static void die (const char *failure) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *failure)
 {

diff --git a/bus/test-system.c b/bus/test-system.c
index 7d5c3dfbd490e40c38798328561fe2d4fff5a624..de1f003b63b4b4b698ba3c77b88527b801a5fc36 100644 (file)
--- a/bus/test-system.c
+++ b/bus/test-system.c
@@ -33,6 +33,8 @@
 #error This file is only relevant for the embedded tests on Unix
 #endif
 
+static void die (const char *failure) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *failure)
 {

diff --git a/dbus/dbus-spawn.c b/dbus/dbus-spawn.c
index 0be8800c329768bea6ef7c78e08767db9976a52b..655bfe92d8d9a07e36048aad945577f78139307e 100644 (file)
--- a/dbus/dbus-spawn.c
+++ b/dbus/dbus-spawn.c
@@ -976,6 +976,8 @@ do_write (int fd, const void *buf, size_t count)
     goto again;
 }
 
+static void write_err_and_exit (int fd, int msg) _DBUS_GNUC_NORETURN;
+
 static void
 write_err_and_exit (int fd, int msg)
 {
@@ -996,6 +998,8 @@ write_pid (int fd, pid_t pid)
   do_write (fd, &pid, sizeof (pid));
 }
 
+static void write_status_and_exit (int fd, int status) _DBUS_GNUC_NORETURN;
+
 static void
 write_status_and_exit (int fd, int status)
 {
@@ -1007,6 +1011,12 @@ write_status_and_exit (int fd, int status)
   exit (0);
 }
 
+static void do_exec (int                       child_err_report_fd,
+                     char             * const *argv,
+                     char             * const *envp,
+                     DBusSpawnChildSetupFunc   child_setup,
+                     void                     *user_data) _DBUS_GNUC_NORETURN;
+
 static void
 do_exec (int                       child_err_report_fd,
         char             * const *argv,
@@ -1125,6 +1135,9 @@ babysit_signal_handler (int signo)
       goto again;
 }
 
+static void babysit (pid_t grandchild_pid,
+                     int   parent_pipe) _DBUS_GNUC_NORETURN;
+
 static void
 babysit (pid_t grandchild_pid,
          int   parent_pipe)

diff --git a/dbus/dbus-test.c b/dbus/dbus-test.c
index c0c6856a79a944246081319cfc18a41257a48414..6c0d71f635e4a89f0341f96f84742bddec3ac2dc 100644 (file)
--- a/dbus/dbus-test.c
+++ b/dbus/dbus-test.c
@@ -29,6 +29,8 @@
 #include <stdlib.h>
 
 #ifdef DBUS_ENABLE_EMBEDDED_TESTS
+static void die (const char *failure) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *failure)
 {

diff --git a/test/name-test/test-ids.c b/test/name-test/test-ids.c
index f73a505fc393b297c99b414ca5ad6a73ceeb75a7..73b71f154349ec29a9f83f0e7ba5ebb7aa3914ce 100644 (file)
--- a/test/name-test/test-ids.c
+++ b/test/name-test/test-ids.c
@@ -8,6 +8,8 @@
 #include <unistd.h>
 #endif
 
+static void die (const char *message) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *message)
 {

diff --git a/test/name-test/test-privserver-client.c b/test/name-test/test-privserver-client.c
index 4268516a17ddee53058e7fa921345c42b141fb68..7a293f4ea5b9158c35f712c2a09a732928797c82 100644 (file)
--- a/test/name-test/test-privserver-client.c
+++ b/test/name-test/test-privserver-client.c
@@ -1,6 +1,9 @@
 #include <config.h>
 #include "../test-utils.h"
 
+static void die (const char *message,
+                 ...) _DBUS_GNUC_NORETURN _DBUS_GNUC_PRINTF (1, 2);
+
 static void
 die (const char *message, ...)
 {

diff --git a/test/name-test/test-privserver.c b/test/name-test/test-privserver.c
index 8bdd4f70b5f487349a25980b9a4c398f3da74f65..535039d8021bd1cbaf768001c7ff80f46d8044fe 100644 (file)
--- a/test/name-test/test-privserver.c
+++ b/test/name-test/test-privserver.c
@@ -1,6 +1,9 @@
 #include <config.h>
 #include "../test-utils.h"
 
+static void die (const char *message,
+                 ...) _DBUS_GNUC_NORETURN _DBUS_GNUC_PRINTF (1, 2);
+
 static void
 die (const char *message, ...)
 {

diff --git a/test/name-test/test-shutdown.c b/test/name-test/test-shutdown.c
index 39612f185dd3c15b62cb47a701fab11fcf14a738..e70bfd39ef95f66bb614076f119d7a3c358adda5 100644 (file)
--- a/test/name-test/test-shutdown.c
+++ b/test/name-test/test-shutdown.c
@@ -4,6 +4,8 @@
 
 static DBusLoop *loop;
 
+static void die (const char *message) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *message)
 {

diff --git a/test/test-names.c b/test/test-names.c
index 273663508533816efd99f8d4e653b64efa2b3458..95e428651491b0ca2bf6e02ce3015328841881e6 100644 (file)
--- a/test/test-names.c
+++ b/test/test-names.c
@@ -3,6 +3,8 @@
 
 static DBusLoop *loop;
 
+static void die (const char *message) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *message)
 {

diff --git a/test/test-service.c b/test/test-service.c
index 564875b057c0f3c5d74e200a53e255be643bc432..bcfb7342c809541b76e466d4022053f6fd5f6284 100644 (file)
--- a/test/test-service.c
+++ b/test/test-service.c
@@ -19,6 +19,8 @@ quit (void)
     }
 }
 
+static void die (const char *message) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *message)
 {

diff --git a/test/test-shell-service.c b/test/test-shell-service.c
index 32a883296e43c129ee11624ffa0ab54de5fb945a..2eaccc1dc4296fa559e4ba18f5d5eb30f64884c6 100644 (file)
--- a/test/test-shell-service.c
+++ b/test/test-shell-service.c
@@ -22,6 +22,8 @@ quit (void)
     }
 }
 
+static void die (const char *message) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *message)
 {

diff --git a/test/test-utils-glib.c b/test/test-utils-glib.c
index e0849771434c73e5cda4be518cdf59a484d877e1..ee286b983a4ef977b75b8dd4c055836d3ac865c8 100644 (file)
--- a/test/test-utils-glib.c
+++ b/test/test-utils-glib.c
@@ -429,6 +429,8 @@ time_out (gpointer data)
 }
 
 #ifdef G_OS_UNIX
+static void wrap_abort (int signal) _DBUS_GNUC_NORETURN;
+
 static void
 wrap_abort (int signal)
 {

diff --git a/test/test-utils.c b/test/test-utils.c
index cb6cf1fb71f36d673d145b53aa181ec65c130487..3f4d195904cf90242761899dafaaaeb2bfbcba30 100644 (file)
--- a/test/test-utils.c
+++ b/test/test-utils.c
@@ -151,6 +151,8 @@ test_connection_setup (TestMainContext *ctx,
   return FALSE;
 }
 
+static void die (const char *message) _DBUS_GNUC_NORETURN;
+
 static void
 die (const char *message)
 {

diff --git a/tools/dbus-echo.c b/tools/dbus-echo.c
index 738b1cb3accb06d64b810a2fdead948ad62eb30d..917d983c6e93aa4154b0b49a33981f90a5b9e26c 100644 (file)
--- a/tools/dbus-echo.c
+++ b/tools/dbus-echo.c
@@ -40,6 +40,8 @@ static int sleep_ms = -1;
 static dbus_bool_t noreply = FALSE;
 static dbus_bool_t noread = FALSE;
 
+static void usage_echo (int exit_with) _DBUS_GNUC_NORETURN;
+
 static void
 usage_echo (int exit_with)
 {
@@ -60,6 +62,8 @@ usage_echo (int exit_with)
   exit (exit_with);
 }
 
+static void usage_black_hole (int exit_with) _DBUS_GNUC_NORETURN;
+
 static void
 usage_black_hole (int exit_with)
 {

diff --git a/tools/dbus-launch.c b/tools/dbus-launch.c
index 4c872c3e548144550b21200d13194a5e2d3a2e23..65287af67164daa00a8d4c01bebfd35eaeda8281 100644 (file)
--- a/tools/dbus-launch.c
+++ b/tools/dbus-launch.c
@@ -164,6 +164,8 @@ verbose (const char *format,
 #endif /* DBUS_ENABLE_VERBOSE_MODE */
 }
 
+static void usage (int ecode) _DBUS_GNUC_NORETURN;
+
 static void
 usage (int ecode)
 {
@@ -174,6 +176,8 @@ usage (int ecode)
   exit (ecode);
 }
 
+static void version (void) _DBUS_GNUC_NORETURN;
+
 static void
 version (void)
 {
@@ -470,6 +474,8 @@ signal_handler (int sig)
     }
 }
 
+static void kill_bus_when_session_ends (void) _DBUS_GNUC_NORETURN;
+
 static void
 kill_bus_when_session_ends (void)
 {
@@ -607,6 +613,7 @@ kill_bus_when_session_ends (void)
             }
         }
     }
+  /* not reached */
 }
 
 _DBUS_GNUC_NORETURN static void
@@ -736,6 +743,12 @@ do_close_stderr (void)
     }
 }
 
+static void pass_info (const char *runprog, const char *bus_address,
+                       pid_t bus_pid, long bus_wid, int c_shell_syntax,
+                       int bourne_shell_syntax, int binary_syntax,
+                       int argc, char **argv,
+                       int remaining_args) _DBUS_GNUC_NORETURN;
+
 static void
 pass_info (const char *runprog, const char *bus_address, pid_t bus_pid,
            long bus_wid, int c_shell_syntax, int bourne_shell_syntax,

diff --git a/tools/dbus-launch.h b/tools/dbus-launch.h
index 538278705197550772c2a44745e6550f4dfad658..56596d26c294c3baacf45cc61584c82828c8e66e 100644 (file)
--- a/tools/dbus-launch.h
+++ b/tools/dbus-launch.h
@@ -42,7 +42,7 @@
 #define MAX_ADDR_LEN 512
 
 /* defined in dbus-launch.c */
-void verbose (const char *format, ...);
+void verbose (const char *format, ...) _DBUS_GNUC_PRINTF (1, 2);
 char *xstrdup (const char *str);
 void kill_bus_and_exit (int exitcode) _DBUS_GNUC_NORETURN;
 

diff --git a/tools/dbus-monitor.c b/tools/dbus-monitor.c
index 483d42eecc7181c474941e47bfcfa7970aba10ac..aa28447fc6b4025dddf8771fe1074a5d961f2b92 100644 (file)
--- a/tools/dbus-monitor.c
+++ b/tools/dbus-monitor.c
@@ -262,6 +262,8 @@ binary_filter_func (DBusConnection *connection,
   return DBUS_HANDLER_RESULT_HANDLED;
 }
 
+static void usage (char *name, int ecode) _DBUS_GNUC_NORETURN;
+
 static void
 usage (char *name, int ecode)
 {

diff --git a/tools/dbus-run-session.c b/tools/dbus-run-session.c
index 105ab3b4fef5b983b33abf0c9c671fd8abebbf2d..135f5d21bf6e9bec5a1ec4a20283b1c9e6c4428e 100644 (file)
--- a/tools/dbus-run-session.c
+++ b/tools/dbus-run-session.c
@@ -65,6 +65,8 @@
 
 static const char me[] = "dbus-run-session";
 
+static void usage (int ecode) _DBUS_GNUC_NORETURN;
+
 static void
 usage (int ecode)
 {
@@ -81,6 +83,8 @@ usage (int ecode)
   exit (ecode);
 }
 
+static void version (void) _DBUS_GNUC_NORETURN;
+
 static void
 version (void)
 {
@@ -95,6 +99,8 @@ version (void)
   exit (0);
 }
 
+static void oom (void) _DBUS_GNUC_NORETURN;
+
 static void
 oom (void)
 {
@@ -191,6 +197,8 @@ exec_dbus_daemon (const char *dbus_daemon,
            me, dbus_daemon, strerror (errno));
 }
 
+static void exec_app (int prog_arg, char **argv) _DBUS_GNUC_NORETURN;
+
 static void
 exec_app (int prog_arg, char **argv)
 {

diff --git a/tools/dbus-send.c b/tools/dbus-send.c
index e77e8098dfe1cadfba09f022a81d454c8b825b52..4982bd49a77c66414db43ac3d550b74e8fdf6aa6 100644 (file)
--- a/tools/dbus-send.c
+++ b/tools/dbus-send.c
@@ -49,6 +49,8 @@
 
 static const char *appname;
 
+static void usage (int ecode) _DBUS_GNUC_NORETURN;
+
 static void
 usage (int ecode)
 {

diff --git a/tools/dbus-spam.c b/tools/dbus-spam.c
index 05d7efa42dab3cc041c8fc26fec20c6347cbd325..9d2b04c3c49c07c63586d548d63fcbebbd640efd 100644 (file)
--- a/tools/dbus-spam.c
+++ b/tools/dbus-spam.c
@@ -35,6 +35,8 @@
 
 static dbus_bool_t ignore_errors = FALSE;
 
+static void usage (int ecode) _DBUS_GNUC_NORETURN;
+
 static void
 usage (int ecode)
 {

diff --git a/tools/dbus-uuidgen.c b/tools/dbus-uuidgen.c
index 744bbeb0e3a0aca63101aa611e3e7d3ba63e1bb8..337bdd0b38830bc9543b6ef94a4f0d26f6c3e82b 100644 (file)
--- a/tools/dbus-uuidgen.c
+++ b/tools/dbus-uuidgen.c
@@ -27,6 +27,8 @@
 #include <dbus/dbus-uuidgen.h>
 #include <dbus/dbus.h>
 
+static void usage (const char *name, int ecode) _DBUS_GNUC_NORETURN;
+
 static void
 usage (const char *name,
        int ecode)
@@ -38,6 +40,8 @@ usage (const char *name,
   exit (ecode);
 }
 
+static void version (void) _DBUS_GNUC_NORETURN;
+
 static void
 version (void)
 {