Re: [systemd-devel] [PATCH] configure: turn off -Wlogical-not-parentheses
Hi On Mon, Feb 16, 2015 at 11:02 PM, Shawn Landden sh...@churchofgit.com wrote: Introduced in gcc-5 These errors are really annoying. I can get behind clarification of nested ifs, but this is overkill. --- configure.ac | 1 + 1 file changed, 1 insertion(+) diff --git a/configure.ac b/configure.ac index 97a29d6..e646db7 100644 --- a/configure.ac +++ b/configure.ac @@ -187,6 +187,7 @@ CC_CHECK_FLAGS_APPEND([with_cflags], [CFLAGS], [\ -Wno-unused-parameter \ -Wno-missing-field-initializers \ -Wno-unused-result \ +-Wno-logical-not-parentheses \ -Werror=overflow \ -Wdate-time \ -Wnested-externs \ I fixed the -Wno-* detection today. Is this patch still needed? I don't have gcc5 here, but I think Daniel fixed most of those warnings recently and they turned out to be real bugs. So should I still apply it? Thanks David -- 2.2.1.209.g41e5f3a ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel
Re: [systemd-devel] [PATCH] configure: turn off -Wlogical-not-parentheses
On Mon, Feb 16, 2015 at 02:02:17PM -0800, Shawn Landden wrote: Introduced in gcc-5 These errors are really annoying. I can get behind clarification of nested ifs, but this is overkill. --- configure.ac | 1 + 1 file changed, 1 insertion(+) diff --git a/configure.ac b/configure.ac index 97a29d6..e646db7 100644 --- a/configure.ac +++ b/configure.ac @@ -187,6 +187,7 @@ CC_CHECK_FLAGS_APPEND([with_cflags], [CFLAGS], [\ -Wno-unused-parameter \ -Wno-missing-field-initializers \ -Wno-unused-result \ +-Wno-logical-not-parentheses \ -Werror=overflow \ -Wdate-time \ -Wnested-externs \ This does not work, gcc does not allow testing for -Wno- options. Zbyszek ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel
Re: [systemd-devel] [PATCH] configure: turn off -Wlogical-not-parentheses
On Mon, Feb 16, 2015 at 2:03 PM, Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl wrote: On Mon, Feb 16, 2015 at 02:02:17PM -0800, Shawn Landden wrote: Introduced in gcc-5 These errors are really annoying. I can get behind clarification of nested ifs, but this is overkill. --- configure.ac | 1 + 1 file changed, 1 insertion(+) diff --git a/configure.ac b/configure.ac index 97a29d6..e646db7 100644 --- a/configure.ac +++ b/configure.ac @@ -187,6 +187,7 @@ CC_CHECK_FLAGS_APPEND([with_cflags], [CFLAGS], [\ -Wno-unused-parameter \ -Wno-missing-field-initializers \ -Wno-unused-result \ +-Wno-logical-not-parentheses \ -Werror=overflow \ -Wdate-time \ -Wnested-externs \ This does not work, gcc does not allow testing for -Wno- options. That is a good thing as otherwise it would break older compilers. This fixes the warnings for me. Zbyszek ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel -- Shawn Landden ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel