[pulseaudio-discuss] [PATCH 0/8] misc cleanups and bug fixes

Peter Meerwald pmeerw at pmeerw.net
Mon Aug 18 06:41:18 PDT 2014


> patch 1 puts *.log and *.trs (which are created by running 'make check') into src/.gitignore
> patch 2 fixes spelling (canceler -> canceller)
> patch 3 add error handling code to pa_srbchannel_new() when pa_fdsem_open_shm() (which it does without HAVE_SYS_EVENTFD_H, e.g. on kFreeBSD)
> patch 4 fix setup_srbchannel() when pa_srbchannel_new() fails
> patch 5 fixes format specifiers, avoid compilation warnings
> patch 6 fixes compilation warning when certain syscalls are not available (e.g. on kFreeBSD)
> patch 7 adds checks for the expected results in sigbus-test
> patch 8 adds a SIGSEGV to the memtrap signal handler to make the sigbus-test work on kFreeBSD

I applied patches 1,2 and 5,6,7,8 (trivial and/or confirmed by Felipe's 
Debian builds)

patches 3,4 will be reposted based on David's comments

> Peter Meerwald (8):
>   tests: Add *.log and *.trs to src/.gitignore
>   echo-cancel: Fix spelling of canceller
>   srbchannel: Cleanup when pa_fdsem_open_shm() fails
>   srbchannel: pa_srbchannel_new() may fail
>   misc: Fix format specifiers
>   core-util: Avoid warnings when missing certain system calls
>   tests: Check expected result of sigbus-test
>   memtrap: Debian/kFreeBSD seems to signal SIGSEGV, not SIGBUS
> 
>  src/.gitignore                               |  2 ++
>  src/modules/echo-cancel/module-echo-cancel.c |  6 ++---
>  src/modules/module-combine-sink.c            |  2 +-
>  src/pulsecore/core-util.c                    | 28 +++++++++++++++-------
>  src/pulsecore/memtrap.c                      |  3 +++
>  src/pulsecore/protocol-native.c              |  6 ++++-
>  src/pulsecore/srbchannel.c                   | 36 ++++++++++++++++++++++++++++
>  src/tests/mcalign-test.c                     |  2 +-
>  src/tests/sigbus-test.c                      |  2 ++
>  9 files changed, 73 insertions(+), 14 deletions(-)
> 
> 

-- 

Peter Meerwald
+43-664-2444418 (mobile)


More information about the pulseaudio-discuss mailing list