meson: Add -Werror=return-type when supported.
authorKenneth Graunke <kenneth@whitecape.org>
Tue, 30 Jan 2018 09:32:07 +0000 (01:32 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 19 Oct 2018 17:16:57 +0000 (10:16 -0700)
commitf91f9bab83dd87034acc520d51d2f9f84984cec6
tree6ad775963e9b44c0c5c9e4b214cac42c9dd07397
parent0d380af8097cf85122596041af1a611a8139de18
meson: Add -Werror=return-type when supported.

This warning detects non-void functions with a missing return statement,
return statements with a value in void functions, and functions with an
bogus return type that ends up defaulting to int.  It's already enabled
by default with -Wall.  Generally, these are fairly serious bugs in the
code, which developers would like to notice and fix immediately.  This
patch promotes it from a warning to an error, to help developers catch
such mistakes early.

I would not expect this warning to change much based on the compiler
version, so hopefully it won't become a problem for packagers/builders.

See the GCC documentation or 'man gcc' for more details:
https://gcc.gnu.org/onlinedocs/gcc-7.3.0/gcc/Warning-Options.html#index-Wreturn-type

Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
meson.build