Mesa (master): meson: add -Werror=empty-body to disallow `if(x);`

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 8d43e2b2ded0fe3c82d49561cdab9f208f9e64b6
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8d43e2b2ded0fe3c82d49561cdab9f208f9e64b6

Author: Eric Engestrom <eric.engestrom at intel.com>
Date:   Mon Sep 23 17:21:20 2019 +0100

meson: add -Werror=empty-body to disallow `if(x);`

This would have prevented a bug in MR 2058 [1]; with that MR fixed,
nothing else uses empty-body blocks, so let's just forbid them altogether.

[1] https://gitlab.freedesktop.org/mesa/mesa/merge_requests/2058#note_237880

Signed-off-by: Eric Engestrom <eric.engestrom at intel.com>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>

---

 meson.build | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meson.build b/meson.build
index b4cd7999566..ac864819013 100644
--- a/meson.build
+++ b/meson.build
@@ -939,6 +939,7 @@ else
     '-Werror=implicit-function-declaration',
     '-Werror=missing-prototypes',
     '-Werror=return-type',
+    '-Werror=empty-body',
     '-Werror=incompatible-pointer-types',
     '-Wno-missing-field-initializers',
     '-Wno-format-truncation',
@@ -958,6 +959,7 @@ else
 
   _trial = [
     '-Werror=return-type',
+    '-Werror=empty-body',
     '-Wno-non-virtual-dtor',
     '-Wno-missing-field-initializers',
     '-Wno-format-truncation',