[PATCH 0/1] MR7753: configure.ac: avoid problems with -Werror=return-type in check

[Marcus Meissner (＠msmeissn)]

Reported-By: Alexander Leithner 
Signed-off-by: Marcus Meissner

[PATCH 1/1] configure.ac: avoid problems with -Werror=return-type in check

[Marcus Meissner]

From: Marcus Meissner

Reported-By: Alexander Leithner 
Signed-off-by: Marcus Meissner 
---
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index a6c048132d4..d919b9dc30f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1997,7 +1997,7 @@ then
         *) AC_CACHE_CHECK([for working -mabi=ms], ac_cv_mabi_ms,
                [CFLAGS="$CFLAGS -mabi=ms"
                 AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <stdarg.h>
-int a(int b, ...) { __builtin_ms_va_list list; __builtin_ms_va_start(list,b); }]])],
+int a(int b, ...) { __builtin_ms_va_list list; __builtin_ms_va_start(list,b); return 0; }]])],
                    [ac_cv_mabi_ms=yes],[ac_cv_mabi_ms=no])
                 CFLAGS=$saved_CFLAGS])
            AS_VAR_IF([ac_cv_mabi_ms],[yes],

[Dmitry Timoshkov (＠dmitry)]

Dmitry Timoshkov (@dmitry) commented about configure.ac:
>      *) AC_CACHE_CHECK([for working -mabi=ms], ac_cv_mabi_ms,
>             [CFLAGS="$CFLAGS -mabi=ms"
>              AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <stdarg.h>
>
> -int a(int b, ...) { __builtin_ms_va_list list; __builtin_ms_va_start(list,b); }]])],
> +int a(int b, ...) { __builtin_ms_va_list list; __builtin_ms_va_start(list,b); return 0; }]])],

Wouldn't it be cleaner to use void return type?