ACK: [PATCH] Makfile: add more warning flags to ensure we catch more issues

ivanhu ivan.hu at canonical.com
Fri Mar 20 03:34:17 UTC 2020



On 3/19/20 8:49 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Increase the build warning checks to catch bad code. This builds
> with gcc 9.3, gcc 10.0, clang 9 and clang 10 without issues.
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/Makefile.am | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 904fc020..8aacff02 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -15,7 +15,13 @@ AM_CPPFLAGS = \
>  	`pkg-config --silence-errors --cflags json-c`	\
>  	-pthread `pkg-config --cflags glib-2.0 gio-2.0` \
>  	-Wall -Werror -Wextra				\
> -	-Wno-address-of-packed-member
> +	-Wno-address-of-packed-member			\
> +	-Wfloat-equal -Wmissing-declarations		\
> +	-Wmissing-format-attribute -Wno-long-long	\
> +	-Wredundant-decls -Wshadow 			\
> +	-Wno-missing-field-initializers 		\
> +	-Wno-missing-braces -Wno-sign-compare 		\
> +	-Wno-multichar -Wcast-qual
>  
>  bin_PROGRAMS = fwts
>  
> 

Acked-by: Ivan Hu <ivan.hu at canonical.com>



More information about the fwts-devel mailing list