bluetooth: destruct stream only if it is not already destructed

https://bugzilla.redhat.com/show_bug.cgi?id=551842

core: make sure we always return a valid memblock in sink_input_pop() callbacks

https://bugzilla.redhat.com/show_bug.cgi?id=553607

native: fix request counter miscalculations

Do not subtract bytes the client sends us beyond what we requested from
our missing bytes counter.

This was mostly a thinko that caused servers asking for too little data
when the client initially sent more data than requested, because that
data sent too much was accounted for twice.

This commit fixes this miscalculation.

http://bugzilla.redhat.com/show_bug.cgi?id=534130

dbus: remove filter functions only if they were actually set before

This fixes an assert when destructing modules that have not been fully
initialized.

https://bugzilla.redhat.com/show_bug.cgi?id=548525

bump soname

build-sys: bump soname

core: Fix macro typo - PA_SINK_IS_LINKED -> PA_SINK_INPUT_IS_LINKED

stream-restore: Clear the save_sink/save_source flags on apply_entry.

If the user specifically removes the device element from the stream
restore rule, we have to clear the save_sink/save_source flag of the
stream. This means that other stream routing systems
(e.g. module-device-manager) can take over routing for this
stream. In order to facilitate the reapplication of other routing
rules, we fire a stream change event. Arguably the stream itself
has not changed, but the rules governing its routing have, so
I feel this is justified.

osx: don't build the once-test binary on OS X

OS X lacks the barrier pthread APIs

client.conf.in: Typo. s/a/are/

This is the same typo as reported in [1] and fixed in [2] in `client.conf.in`.

I did `git grep "values a commented"` and did not get anymore hits.

[1] http://pulseaudio.org/ticket/783
[2] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=ea8927f1d47be101a1f64b2f87fd6e527ff76dbe

man pages: correct formatting/markup of options

shm: explicitly mark shm seg for MAP_NORESERVE to request overcommiting no matter what

Typo. s/a/are/

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>

client: include dolby channel names in comments

start: we don't need to check for $PULSE_SERVER anymore

pulseaudio --start does that internally anyway, so we can get rid of
this here.

native: when run in system mode, do not look for fallback port

native: fallback to another port if the default port is taken

Unless the port number is explicitly configured we will now fallback to
a kernel picked port if the one we'd like by default we cannot get.

http://pulseaudio.org/ticket/773

client: introduce auto-connect-display= following the scheme of auto-connect-localhost=

Just connecting to a PA server just because X11's $DISPLAY is set might
be a security hole.

client: introduce auto-connect-localhost= option in client.conf

Partly fixes:

http://pulseaudio.org/ticket/773

Also fixes a security hole since listening on the default port is not
access controlled right now.

Merge remote branch 'tanuk/fixes'

tests: add pa_once_xxx() test

daemon: Don't autospawn if a server address is explicitly configured.

sink-input: Replace a tab indentation with spaces.

idxset: Fix _get_by_data() comment.

core: volume ramping fix

Hi,
I found that volume ramping is inside PA now.
there's a minor fix here, it is a bug i found after the patch is submitted:

line 1781 of sink-input.c :
if ((i->thread_info.ramp_info.envelope_dying - nbytes) <= 0) {
need to be changed to
if ((i->thread_info.ramp_info.envelope_dying - (ssize_t) nbytes) <= 0) {

otherwise this argument will never be negative since nbytes is of type
size_t which is unsigned.
Please change it when you have time, sorry if bring any inconvenience.  :)

Merge remote branch 'zonque/topic/osx'

fix a number of warnings

most of them were due to missing #ifdefs or wrong printf format type for
[s]size_t.

headers: Some trivial fixes for some documentation typos

Note also the willneed/will_need inconsistency. I guess it could be nice to ASAP
choose one of them and introduce a backward compatibility hack for the other.

The issues was mostly found with:
for a in $(grep -r '^[ /]\*.*()' $(
find -name '*.[ch]') |
sed 's,^.* \([^ ]*\)().*$,\1,g' |
sort |
uniq |
grep ^pa_)
do
grep -rq "^.[^*].*\<$a(" $(find * -name '*.h') || echo $a
done

Subject: rtpoll: better support for DEBUG_TIMING logs

On all the platforms I tested, PulseAudio is frequently awaken and
doesn't sleep for the duration specified for the poll timeout.
Sometimes wake-ups occur within milliseconds of the poll call for no
good reason; this seems to be related to ALSA issues (see my posts on
the ALSA mailing list on null poll events).
This patch enables a better log of requested sleep times v. actual
sleep times. Enable DEBUG_TIMING to see actual messages. Please let me
know if you see odd behaviors like the one below
- Pierre

E: rtpoll.c: rtpoll_run
E: rtpoll.c: poll timeout: 188 ms
E: rtpoll.c: Process time 0 ms; sleep time 48 ms
E: rtpoll.c: rtpoll_run
E: rtpoll.c: rtpoll finish
E: rtpoll.c: rtpoll_run
E: rtpoll.c: poll timeout: 139 ms
E: rtpoll.c: Process time 0 ms; sleep time 49 ms
E: rtpoll.c: rtpoll_run
E: rtpoll.c: rtpoll finish
E: rtpoll.c: rtpoll_run
E: rtpoll.c: poll timeout: 189 ms
E: rtpoll.c: Process time 0 ms; sleep time 0 ms
E: rtpoll.c: rtpoll_run
E: rtpoll.c: rtpoll finish
E: rtpoll.c: rtpoll_run
E: rtpoll.c: poll timeout: 189 ms
E: rtpoll.c: Process time 0 ms; sleep time 49 ms

Fix typo in log message: s/may no be/may not be/

Hi, Lennart.

Spotted another typo. Patch attached.

Best wishes,
chocolateboy.

alsa: add profile set for M-Audio FastTrack Pro USB

udev: Use SOUND_CLASS instead of SOUND_FORM_FACTOR when checking for modem

The fact whether an ALSA card is a modem is stored in the SOUND_CLASS,
not the SOUND_FORM_FACTOR property. So read it from there.

Patch from Whoopie.

threaded-mainloop: Properly initialise m->n_waiting_for_accept to prevent deadlock

Compiler optimisations have been seen to initialise
m->n_waiting_for_accept to a positive non-zero value, so the while() in
pa_threaded_mainloop_signal() never proceeds. Fix this by properly
initializing m->n_waiting_for_accept in pa_threaded_mainloop_new().

Patch from Iain Bucław.

https://bugs.launchpad.net/bugs/502992

More src/pulsecore/cpu-arm.c FTBFS fixes

Fix missing argument to pa_read(), and be consistent with declaration of
state variable in pa_cpu_init_arm().

Signed-off-by: Daniel T Chen <crimsun@ubuntu.com>

Fix the following warnings (which now cause buildd failures in Ubuntu 10.04):

pulsecore/cpu-arm.c: In function 'get_cpuinfo':
pulsecore/cpu-arm.c:70: warning: implicit declaration of function 'pa_read' [-Wimplicit-function-declaration]
pulsecore/cpu-arm.c:72: warning: implicit declaration of function 'pa_close' [-Wimplicit-function-declaration]
pulsecore/cpu-arm.c: In function 'pa_cpu_init_arm':
pulsecore/cpu-arm.c:110: warning: implicit declaration of function 'pa_split_spaces' [-Wimplicit-function-declaration]
pulsecore/cpu-arm.c:110: warning: assignment makes pointer from integer without a cast
Function `pa_split_spaces' implicitly converted to pointer at pulsecore/cpu-arm.c:110

Signed-off-by: Daniel T Chen <crimsun@ubuntu.com>

fix a number of warnings

most of them were due to missing #ifdefs or wrong printf format type for
[s]size_t.

osx: add native zeroconf implementation via Bonjour

Avahi and dbus is too heavy for OSX just for the sake of publishing our
services via mDNS/Zeroconf. Apple has its own Zeroconf implementation
called Bonjour, and this patch adds a module that implements service
announcement with that API.

All data gathering is copied from module-zeroconf-publish.c, but
unfortunately the code there is too specifically made for avahi, so I
couldn't factor it out to reuse it.

CoreAudio: add audio device module

This patch adds support for CoreAudio driven devices under Mac OS X. It
is typically instanciated by the CoreAudio device detection module and
handles all available streams on a specific device.

Sinks are created according to the reported stream configuration.
Float32 is used as default audio sample format at it is the only format
CoreAudio speaks natively.

Hardware volume control is not implemented yet.

CoreAudio: add device detection module

This adds a new module for CoreAudio device detection. It registers a
callback to detect hotplugged devices and creates/destroys modules named
'module-coreaudio-device'. Devices are identified via a system-wide
unique AudioDeviceID.

hack around another OS X bug: recv() with MSG_PEEK does not work

At least for pipes, recv() with MSG_PEEK does actually eat up data from
file descriptors. Hence, this can't be used for PULLHUP emulation.

Use another ioctl hack for that.

poll() is totally broken on Mac OS X

Even on 10.5.8, poll() does not do the right thing. Haven't checked on
newer versions. Hence, wrap all occurences of poll() to pa_poll and
emulate that call with select() on OSX. This is totally embarassing.

src/Makefile.am: add specific OS_IS_DARWIN files

Signed-off-by: Kim Lester <kim@dfusion.com.au>
Signed-off-by: Daniel Mack <daniel@caiaq.de>

core-rtclock.c: tweak OS_IS_DARWIN constraints

Move the code for OS_IS_DARWIN to the top as on Darwin,
HAVE_CLOCK_GETTIME is also defined.

configure.ac: add DARWIN_OS variable

Signed-off-by: Kim Lester <kim@dfusion.com.au>
Signed-off-by: Daniel Mack <daniel@caiaq.de>

Merge branch 'master' of git://0pointer.de/pulseaudio

Add a configure option to change 'udevrulesdir'

This patch serves two purposes:

  1) Allows something other than the de-facto standard udev rules dir or
     /lib/udev/rules.d to be used (the udev build system allows you to
     customise this)

  2) Allows a prefixed, non-root install (right now, the /lib/... path
     is hard-coded into the build system

Mark shared variables as volatile

'n_waiting' and 'n_waiting_for_accept' may be accessed from mulitple
threads, and thus need to be marked as volatile to suppres certain
compiler optimisations. All uses are protected by a mutex, so we don't
need to worry about cache issues (added documentation for this as well).

This addresses bug #738.

stream-restore: At startup, create dbus entries only for valid database entries.

stream-restore: Add a missing pa_xnew0() call in handle_add_entry().

stream-restore: Fix a few assertion misuses with the D-Bus code.

dbus: Add a missing break statement in handle_message_cb().

dbus: Handle the cases when a non-existing interface is detected in an incoming message.

libpulse: Store pa_stream pointers to hashmaps instead of dynarrays.

Since the stream identifiers (channels) are monotonically growing integer, it
isn't a good idea to use them as index to a dynamic array, because the array
will grow all the time. This is not a problem with client connections that
don't create many streams, but, for example, long-running clients that use
libcanberra for playing event sounds, this means that the client connection
effectively leaks memory.

Merge branch 'master' of git://0pointer.de/pulseaudio

Merge remote branch 'tanuk/master'

Merge branch 'master' of git://0pointer.de/pulseaudio

Merge branch 'master' of ssh://rootserver/home/lennart/git/public/pulseaudio

Merge remote branch 'phish3/master'

Merge branch 'master' of ssh://rootserver/home/lennart/git/public/pulseaudio

freebsd: implement pa_get_binary_name

Stolen from
http://www.freebsd.org/cgi/cvsweb.cgi/ports/audio/pulseaudio/files/patch-src_pulse_util.c?rev=1.1
and fixed for indentation.

src/utils/qpaeq: added more friendly error messages to common errors

Makefile.am: added qpaeq to installed scripts

Simplify Makefile.am handling of ALSA-related files.

Instead of declaring extra variables for the ALSA profiles and PATHS, and
using EXTRA_DIST for the udev rule, use the dist_ prefix when declaring
them for conditional installation.

This relies on the fact that automake *is* smart enough to know that there
exist *no* conditional dist, and will always distribute all of them.

Avoid using devname as a variable name.

On FreeBSD, devname() is a system function, and was overshadowed.

Rename all the signal parameters and variables to something more explicit.

Without this change, on FreeBSD you'll be bothered by tons of warnings
about overshadowing signal(2).

Since now we have FreeBSD atomic operations, don't require libatomic_ops.

This lets the original configure script to pass on FreeBSD.

Simplify handling of NetBSD atomic ops discovery.

Instead of having a parameter to enable/disable them (which was also
ignored by the source code side), simply identify NetBSD 5 (when the atomic
ops were implemented the first time) in a $host switch and ignore checking
for libatomic_ops.

Merge branch 'master' of git://git.0pointer.de/pulseaudio

remove .py extension from qpaeq

Merge remote branch 'phish3/master'

freebsd: fix atomic ops implementations

Stole from

http://www.freebsd.org/cgi/cvsweb.cgi/ports/audio/pulseaudio/files/patch-src_pulsecore_atomic.h?rev=1.6

added qpaeq script for GUI equalizer control to src/util

Check for stow using AC_CHECK_PROG rather than type -p.

The type -p check is not compatible with POSIX shells, and indeed fails
when used with the FreeBSD 8 default shell. Instead the AC_CHECK_PROG macro
is written properly to look for the command.

Include sys/filio.h if present; this makes use of FIONREAD on Solaris.

FIONREAD on OpenSolaris is declared in sys/filio.h. The RTP modules already
rely on the presence of FIONREAD, and check for the header, so this brings
the pipe source and sink modules on par with them.

Don't declare the variable l if FIONREAD is not defined.

This avoids two warnings due to FIONREAD not being defined.

Fix build on Solaris: pass the third parameter to pa_cloexec_open.

Move the platform-specific defines after the compiler has been found.

Without this change, on OpenSolaris, it's possible that the compiler will
be reported, by default, as failing to produce executables, since
_XOPEN_SOURCE=600 requires a C99 compiler to compile even the simples of
the programs.

alsa: fix minor sampling rate deviations before adjusting the buffer size

alsa: fix log output when the audio device refuses to give us again the same period settings we had before

pulse: ask for timing updates both *before* and *after* triggering a stream state change so that in the STARTED/UNDERFLOW callbacks we accurate transport latency information

pulse: delay smoother update only when unpausing, not when pausing, since we don't want the timer to advance when we are supposedly already paused

pulse: try to fix inaccuracy with uncork timing for streams that are created in corked state

daemon: complain if user passes too many arguments

module-equalizer-sink: add latency of output_q and input_q to get latency calculation

module-equalizer-sink: switch back to reference dsp implementation - cpu usage doesn't really change and there may be
a bug in the vectorized version

module-equalizer-sink: (re)added output memblockq commented out timing debug statements

Merge branch 'master' of ssh://rootserver/home/lennart/git/public/pulseaudio

rygel: fix itemCount property for sources

Fixes http://pulseaudio.org/ticket/721

build-sys: Fix missing trailing slash in 14eaf2

build-sys: Make sure all alsa path config files are installed

While all alsa path configuration files were in EXTRA_DIST not all of them were
in in alsapaths_DATA. Fixed by only defining the list in one place

device-manager: Update docs version -> 0.9.20

Third time is a charm... maybe.

Merge branch 'master' of git://0pointer.de/pulseaudio

core: make cpuid code compile cleanly with 32bit PIC

core: adjust volume only when there is actually a memory block

Fixes an assert that is hit in somne niche cases:

https://bugzilla.redhat.com/show_bug.cgi?id=533482

protocol: use the right sample rate for sources

Use the correct sample rate for reporting about the timing.

device-manager: Update docs version -> 0.9.20

ramping: minor cleanups

Merge remote branch 'origin/merge-queue'

Conflicts:
src/pulsecore/sink-input.c
src/pulsecore/sink.c

jack: never try to autoconnect to MIDI ports

Original patch supplied by 'adi'

http://pulseaudio.org/ticket/669

man: fix build with --disable-manpages

http://pulseaudio.org/ticket/698