On Wed, 2015-12-16 at 09:10 +0530, [email protected] wrote: > From: Arun Raghavan <[email protected]> > > This is needed for building with anonymous unions. A bunch of calls to > fail() that used to mysteriously work need fixing -- fail() is a macro > that takes a printf-style message as an argument. Not passing this > somehow worked with the previous compiler flags, but breaks with > -std=c11. > --- > configure.ac | 3 ++- > src/tests/connect-stress.c | 6 +++--- > src/tests/cpu-mix-test.c | 2 +- > src/tests/cpu-remap-test.c | 4 ++-- > src/tests/cpu-sconv-test.c | 4 ++-- > src/tests/cpu-volume-test.c | 2 +- > src/tests/cpulimit-test.c | 4 ++-- > src/tests/extended-test.c | 4 ++-- > src/tests/get-binary-name-test.c | 2 +- > src/tests/interpol-test.c | 2 +- > src/tests/mult-s16-test.c | 2 +- > src/tests/sync-playback.c | 4 ++-- > 12 files changed, 20 insertions(+), 19 deletions(-) > > diff --git a/configure.ac b/configure.ac > index 26c3e29..6fc77c5 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -80,7 +80,6 @@ AC_PROG_LN_S > # CC > > AC_PROG_CC > -AC_PROG_CC_C99 > AM_PROG_CC_C_O > # Only required if you want the WebRTC canceller -- no runtime dep on > # libstdc++ otherwise > @@ -176,6 +175,8 @@ esac > > #### Compiler flags #### > > +AX_APPEND_COMPILE_FLAGS([-std=c11], [], [-pedantic -Werror])
We discussed this a while back in irc, and my recollection is that we agreed that some macro with "CHECK" in its name seemed the best fit. AX_APPEND_COMPILE_FLAGS is not good, because it silently drops the flag if -std=c11 isn't supported. -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
