Hi Bastien, On Thu, Mar 4, 2021 at 9:21 AM Bastien Nocera <hadess@xxxxxxxxxx> wrote: > > --- > acinclude.m4 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/acinclude.m4 b/acinclude.m4 > index 529848357..6ae34b8ae 100644 > --- a/acinclude.m4 > +++ b/acinclude.m4 > @@ -21,7 +21,7 @@ AC_DEFUN([COMPILER_FLAGS], [ > with_cflags="$with_cflags -Wredundant-decls" > with_cflags="$with_cflags -Wcast-align" > with_cflags="$with_cflags -Wswitch-enum" > - with_cflags="$with_cflags -Wformat -Wformat-security" > + with_cflags="$with_cflags -Wformat -Wformat-security -Wformat-nonliteral" Does it actually have any benefit of having the format as always string literal? I'm not really a big fan of using pragmas. > with_cflags="$with_cflags -DG_DISABLE_DEPRECATED" > with_cflags="$with_cflags -DGLIB_VERSION_MIN_REQUIRED=GLIB_VERSION_2_28" > with_cflags="$with_cflags -DGLIB_VERSION_MAX_ALLOWED=GLIB_VERSION_2_32" > -- > 2.29.2 > -- Luiz Augusto von Dentz
