diff options
| author | Nick Mathewson <nickm@torproject.org> | 2016-05-27 11:26:14 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2016-05-27 11:26:14 -0400 |
| commit | ce1dbbc4fdd7ab6fab4f31a0e54d9ca97afe74a9 (patch) | |
| tree | 354dedd9d603f203aa65b1755138cf939e8216b6 | |
| parent | 0df2c5677a79aafc32591b15ca07d25c638b68cc (diff) | |
| download | tor-ce1dbbc4fdd7ab6fab4f31a0e54d9ca97afe74a9.tar.gz tor-ce1dbbc4fdd7ab6fab4f31a0e54d9ca97afe74a9.zip | |
Enable the -Waggregate-return warning
Suppress it in the one spot in the code where we actually do want to
allow an aggregate return in order to call the mallinfo() API.
| -rw-r--r-- | configure.ac | 3 | ||||
| -rw-r--r-- | src/common/util.c | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/configure.ac b/configure.ac index 29715c1a3d..d922eb636d 100644 --- a/configure.ac +++ b/configure.ac @@ -1698,8 +1698,7 @@ if test "x$enable_gcc_warnings_advisory" != "xno"; then CFLAGS="$CFLAGS -Wredundant-decls -Wchar-subscripts -Wcomment -Wformat=2" CFLAGS="$CFLAGS -Wwrite-strings -Wmissing-declarations -Wredundant-decls" CFLAGS="$CFLAGS -Wnested-externs -Wbad-function-cast -Wswitch-enum" - - # Disabled, so we can use mallinfo(): -Waggregate-return + CFLAGS="$CFLAGS -Waggregate-return" if test "x$have_gcc4" = "xyes"; then # These warnings break gcc 3.3.5 and work on gcc 4.0.2 diff --git a/src/common/util.c b/src/common/util.c index 78afe5954f..925431d181 100644 --- a/src/common/util.c +++ b/src/common/util.c @@ -342,6 +342,7 @@ tor_free_(void *mem) tor_free(mem); } +DISABLE_GCC_WARNING(aggregate-return) /** Call the platform malloc info function, and dump the results to the log at * level <b>severity</b>. If no such function exists, do nothing. */ void @@ -369,6 +370,7 @@ tor_log_mallinfo(int severity) ); #endif } +ENABLE_GCC_WARNING(aggregate-return) /* ===== * Math |