tests: use function attribute consistently

Move all function attributes to the function definitions and put them
before the return type.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

diff --git a/tests/unit/gpiod-unit.c b/tests/unit/gpiod-unit.c
index a6677252e08fe2f0531f0e2eba012650c990edf2..3fa77aa1622d0012d4de6bc5248a1db03dc7bf0c 100644 (file)
--- a/tests/unit/gpiod-unit.c
+++ b/tests/unit/gpiod-unit.c
@@ -52,7 +52,7 @@ static void vmsg(FILE *stream, const char *hdr, const char *fmt, va_list va)
        fputc('\n', stream);
 }
 
-static void GU_PRINTF(1, 2) msg(const char *fmt, ...)
+static GU_PRINTF(1, 2) void msg(const char *fmt, ...)
 {
        va_list va;
 
@@ -61,7 +61,7 @@ static void GU_PRINTF(1, 2) msg(const char *fmt, ...)
        va_end(va);
 }
 
-static void GU_PRINTF(1, 2) err(const char *fmt, ...)
+static GU_PRINTF(1, 2) void err(const char *fmt, ...)
 {
        va_list va;
 
@@ -70,7 +70,7 @@ static void GU_PRINTF(1, 2) err(const char *fmt, ...)
        va_end(va);
 }
 
-static void GU_PRINTF(1, 2) NORETURN die(const char *fmt, ...)
+static GU_PRINTF(1, 2) NORETURN void die(const char *fmt, ...)
 {
        va_list va;
 
@@ -81,7 +81,7 @@ static void GU_PRINTF(1, 2) NORETURN die(const char *fmt, ...)
        exit(EXIT_FAILURE);
 }
 
-static void GU_PRINTF(1, 2) NORETURN die_perr(const char *fmt, ...)
+static GU_PRINTF(1, 2) NORETURN void die_perr(const char *fmt, ...)
 {
        va_list va;
 
@@ -461,7 +461,7 @@ void _gu_register_test(struct _gu_test *test)
        globals.num_tests++;
 }
 
-void _gu_test_failed(const char *fmt, ...)
+GU_PRINTF(1, 2) void _gu_test_failed(const char *fmt, ...)
 {
        va_list va;
 

diff --git a/tests/unit/gpiod-unit.h b/tests/unit/gpiod-unit.h
index 9bae567ce09a821fcff294fd356cec6cdb300575..bd6fe9f87ca5d025e1ad9f950bb15d40b03096fe 100644 (file)
--- a/tests/unit/gpiod-unit.h
+++ b/tests/unit/gpiod-unit.h
@@ -39,7 +39,7 @@ struct _gu_test {
 };
 
 void _gu_register_test(struct _gu_test *test);
-void _gu_test_failed(const char *fmt, ...) GU_PRINTF(1, 2);
+void _gu_test_failed(const char *fmt, ...);
 
 /*
  * This macro should be used for code brevity instead of manually declaring