[pulseaudio-discuss] [PATCH 0/6] Fix compilation warnings with NDEBUG

Peter Meerwald pmeerw at pmeerw.net
Sun Nov 9 08:59:20 PST 2014


fix several warnings when compiling with NDEBUG:
* pa_assert_not_reached() should abort even when compiling with NDEBUG
* variables just used in assert()s trigger a warning, fix this by 
  annotating these variables
* some cleanup, turn remaining assert()s in pa_assert()s

Peter Meerwald (6):
  macro: Abort() when pa_assert_not_reached() even for NDEBUG
  macro: Add macro PA_USED
  core: Annotate variables only used within assert()s to be PA_UNUSED
  tests: Use pa_assert_se() to get rid of an used variable warning
  alsa-mixer: Use pa_assert_not_reached()
  core: Replace assert()s with pa_assert()s

 src/modules/alsa/alsa-mixer.c                |  3 +--
 src/modules/bluetooth/backend-native.c       |  4 ++--
 src/modules/echo-cancel/module-echo-cancel.c |  6 +++---
 src/modules/module-console-kit.c             |  2 +-
 src/modules/module-solaris.c                 |  6 +++---
 src/pulsecore/aupdate.c                      |  2 +-
 src/pulsecore/macro.h                        |  8 +++++++-
 src/pulsecore/memchunk.c                     |  2 +-
 src/pulsecore/pipe.c                         |  2 +-
 src/pulsecore/sink-input.c                   |  6 +++---
 src/pulsecore/sink.c                         |  2 +-
 src/pulsecore/source-output.c                |  2 +-
 src/pulsecore/source.c                       |  2 +-
 src/tests/flist-test.c                       |  2 +-
 src/tests/format-test.c                      |  2 +-
 src/tests/lo-test-util.c                     | 10 ++++------
 src/tests/mcalign-test.c                     |  3 +--
 17 files changed, 33 insertions(+), 31 deletions(-)

-- 
1.9.1



More information about the pulseaudio-discuss mailing list