build-sys: Define PA_CFLAGS at right time

To ensure that all the changes to CFLAGS are also stored into PA_CFLAGS.

build-sys: Reset CFLAGS after DBUS check

build-sys: Consolidate host_os handling

Use os_is_* shell variables instead of pulse_target_os.

build-sys: Remove unnecessary AC_SUBST calls

These HAVE_* variables are only used as AM_CONDITIONAL, so AC_SUBST is not needed.

Disable check for pthreads on win32

And do not use sched_get_priority on mingw with win32 pthreads installed

protocol-native: Use original requested latency on stream moves

We were calculating new latency based on the latency set on the old
sink/source, rather than the actual latency requested by the client.
Over a series of moves, this will lead the latency being ~halved each
time, resulting in an eventual rewind flood from a latency that cannot
be handled.

echo-cancel: More preprocessing fixes

The speex_preprocess_ctl() function takes a spx_int32_t, but we've been
passing a pa_bool_t, which could potentially crash.

echo-cancel: Fix preprocessor initialisation

We were using the block size in bytes instead of samples, which meant
preprocessing was broken. This fix makes a large-ish difference in the
quality of echo-cancellation with speex.

win32: define WIN32_LEAN_AND_MEAN

This makes windows.h include less headers.
Otherwise boolean is typedef'ed and that clashes with libjson.

gitignore: Add connect-stress, extended-test and format-test.

tests: Update extended API test

The test wasn't updated after we changed the pa_format_info proplist
format.

combine: Fix a crash on shutdown if the module is loaded outside of our control.

By using module indexes rather than module pointers we avoid this posibility.

device-manager: Free a hook slot to prevent segv on unload

pulsecore: Add a couple pa_asserts() on pa_tagstruct* calls.

module-combine-sink: Check running flag before rendering in null mode

This makes process_render_null consistent with render_memblock and
avoids introducing slight inaccuracies in early latency estimates.

module-combine-sink: Initialize smoother in paused state

The smoother is paused on initialization and resumed when the sink
state is set to running.  Otherwise, early latency estimates are
too low since there is some delay between module initialization and
entering the running state.

After the smoother is initially resumed, it is paused when the sink
state is not running.  The previous behavior was to pause only when
the sink enters suspended state, however, this would lead to large
errors in latency estimates after the sink has been idle for some
time.

module-combine-sink: Initialize smoother with offset pa_rtclock_now()

The smoother was being initialized with offset zero, which caused
the sink latency to be unconditionally reported as zero.

streams: Fix the actual resampler method shown in debug messages.

osx: pass -headerpad_max_install_names to the linker, too

This option won't make it to the actual libtool command which does the
linking when not prefixed with -Wl,

alsa-mixer: Fix the assumption that volume is always positive

Add a variable to track whether the actual volume is set or not.
Suppose this:
min volume: -126 max volume: 0
then when user wants to set some constant volume to -10, it would fail.

While the alsa values are typically positive, some values are "funky"
and have negative values. It is desirable to fix this at the alsa
level so that the numbers are positive, but it's not technically
invalid, and thus we have to support it.

Discussed here:
http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/9832
and
http://thread.gmane.org/gmane.linux.alsa.devel/85459

echo-cancel: Fix a crash is speex cleanup

If module initialisation fails, the speex done() function might try to
free a value that's not been allocated yet. Adding protection for this
condition.

echo-cancel: Fix echo suppression, add some knobs

The echo suppress attenuation value was being incorrectly modified.
Fixed and added 2 arguments to change the attenuation of the residual
echo filter. Default values of the speex preprocessor will be used when
omitted.

echo-cancel: Speex preprocessor has to run *after* the AEC.

This is how it is expected to be run.

echo-cancel: Add speex preprocessing

This allows the selective enabling of speex' preprocessing algorithms
before running the echo-canceller -- for now this includes automatic
gain control, noise suppression and echo suppression. It's all off by
default for now, though at some point in the near future we might want
to enable at least denoising by default.

The denoising works pretty well, though we might want to add a way to
tweak the noise-suppression knob that libspeex provides.

The AGC option is just a stop-gap -- we need a real AGC mechanism that
tweaks the source volume rather than doing this in software.

The speex documentation mentions VAD and dereverb, but it appears that
these are not complete yet.

We don't do all this in a separate module from module-echo-cancel to
avoid the overhead of adding another virtual source. It makes more sense
to make a separate virtual source module that can be used for cases
where preprocessing is useful but AEC is not (for e.g. noise suppression
for fan noise in a recording application).

Another reason to keep this integrated with the AEC module is that the
echo suppression bits use the speex echo canceller state. This does leak
some information about the AEC implementation into module-echo-cancel,
but this is unavoidable.

format: Fix channel map handling

Channel map handling in the extended API was broken. Thanks for Milos_SD
for pointing this out on IRC.

echo-cancel: Remove extraneous debug message

This was making it impossible to run in debug mode with save_aec=1

echo-cancel: Don't overpad variable

The padding was to be 16 bytes, not 16 elements.

build-sys: Link libpulse directly to libdbus-1 if needed

Linking libpulse with gold or when using ld --no-add-needed fails
as libpulse uses dbus methods directly but isn't explicitly linked to it.
So link to it when needed :)

protocol-native: Fix memory leaks introduced in protocol 21 (passthrough support)

The proplist used may never be freed if an error condition was found with
CHECK_VALIDITY macro and the formats idxset was never freed regardless
of error state.

This change fixes adds a new maco CHECK_VALIDITY_GOTO() which allows
for cleanup to be done before returning.

sink-input: Fix memory leak of proplist when sending format-changed events

loopback: Add a modarg for disabling remixing.

bluetooth: Fix early return styling and add missing return value

Thanks to Tanu Kaskinen for pointing out the missing return.

suspend-on-idle: Trigger mempool vacuuming

In a setup with one or more filter sinks or sources there is always at
least one stream existing. In such a situation normal mempool
vacuuming never happens. This patch causes suspend-on-idle module to
vacuum memory when ever it notices that all sinks and sources are
suspended. The behavior can be enabled with a module parameter.

protocol-native: Stop auto timing updates if connected to suspended sink or source

This quite is an old patch. It was added to N900 to avoid unnecessary
wake-ups when the phone is in power save mode (= blank screen and
no user interaction). In this situation if the user had a browser
window with flash animation open pulseaudio kept waking up every
10 seconds, causing a severe hit to use times.

Anyway I do not see any reason to send timing updates if the sink or
source where the stream is connected to is suspended.

i18n: Fix POTFILES

tests: Fix resampler-test.

This has been broken since c376ac5920fdeb46ca844d9518e22f17adffb635 when run
without any arguments. Passing in -v (verbose) caused the test to work fine.

I think this oversight is just a thinko in the original work but it obviously
broke 'make check' and thus distcheck.

Also fix a couple compiler warnings.

Merge branch 'passthrough'

module-tunnel: Update for recent protocol changes

This updates the tunnel module for protocol version >= 19.

module-tunnel-sink does not proxy server-side passthrough support (yet).
This would require a few more changes, namely keeping track of what
formats are available and if any other sink inputs are connected on the
server-side.

doxygen: generate documentation for format.h

format: Add some convenience API for setting properties

Adds functions to set sample format, rate, channels and channel map on a
format to make life easier for users of the API.

format: Extend properties to handle lists/ranges

This replaces the simple string used by pa_format_info's proplist with a
JSON string (accessed via new API only). This allows us to express lists
and ranges more cleanly, and embed type information for future
extensibility.

We use json-c for JSON parsing. This is a lightweight depdency (32 KB on
my system) and avoids the hassle of having to reinvent a JSON parser.

Also included is a test which verifies functionality and is
valgrind-clean.

sink-input: Provide more information to client when format is lost

When the sink format changes and we kill the stream, clients need a way
to know (a) what device they should reconnect to, and (b) what the
stream running time was when the stream got killed (pa_stream_get_time()
won't work after the stream has been killed). This adds these two bits
of information in the event callback's proplist parameter.

format: Add correct sample spec conversion for E-AC3

IEC61937-encapsulated E-AC3 frames contain 6 audio blocks per substream,
which corresponds to 1536 samples contained a 24576-byte frame. To cope
with this, we maintain the s16le stereo sample spec, but quadruple the
sample rate so that the conversion remains accurate.

format: Export pa_format_info_is_compatible in API

This allows clients to perform checks between formats as well.

sink-input: Add a format-lost event

This event is emitted if the sink-input could not be moved to a new sink
because it doesn't support the format of the sink-input. Clients can
reconnect their stream with a different format if they wish or
gracefully exit.

sink-input: Don't restore volume for passthrough streams

sink-input: Don't print an error if a passthrough connection fails

The assertion message is misleading, since the passthrough connection
can fail for reasons the client has no control over (like other sink
inputs being connected).

echo-cancel: Remove unnecessary noalign attribute

This was just introduced for debugging and should not have been in the
final commit. Won't make a difference at the moment since this function
is used as a pointer, but removing this in case we change this in the
future.

echo-cancel: Handle alignment requirement manually

PA_ALIGNED can't always guarantee that the alignment we want (the GCC
man page suggests that the linker might not be able to meet the
alignment requirements we desire). Instead, we now allocate some extra
memory and guaratee that the alignment we require is met.

rtkit: use private bus connection in order to avoid threading issues when invoking pa_make_realtime()

filter-apply: Mark modules as being autoloaded

(Based on Colin's review) We mark modules as being autoloaded so that
they can handle this as a special case if needed (which is required by
module-echo-cancel for now). This inverts how things were done and makes
using these modules manually less error-prone.

core: Factor out passthrough checks into their own functions

Since we currently have two mechanisms to signal a passthrough
connection (non-PCM format or PA_SINK_INPUT_PASSTHROUGH flag), we move
all the related checks into functions and use those everywhere.

This makes things more consistent, and should we decide to get rid of
the flag, we only need to change pa_sink_input_*_is_passthrough()
accordingly.

format: Add a type for DTS

introspect: Get format of sink input

This gets the negotiated format of sink inputs in
pa_context_get_sink_input*(). Also prints the format in 'pactl list'.

introspect: Get formats for sinks

This gets the list of supported formats for a sink in
pa_context_get_sink_info*(). Also prints these in 'pactl list'.

stream: Add API to get a stream's pa_format_info

format: Add some convenience functions for printing

format: Const-ify some parameters

alsa: Reconfigure sink sample rate for passthrough inputs

When a passthrough sink-input is added, we need to reconfigure the
sink's sample rate since no resampling occurs. We revert to the original
rate when the passthrough sink-input is removed.

core: Suspend monitor when a sink enters passthrough mode

In most cases it is expected that clients cannot consume compressed
data from monitor sources, so we suspend the monitor source when the
sink goes into passthrough mode.

Eventually, when the extended API includes client notifications for
changed formats, we should emit a notification on the monitor so that
clients can decide what they want to do when this happens (disconnect or
consume the data anyway).

sink: Trivial typo fix in comment

alsa-mixer: Remove passthrough profiles

These aren't used any more - we handle passthrough mode in the iec958*
profiles now.

core: Fix some FIXMEs for the extended API

This adds some checks that I'd postponed and adds a
"should-be-good-enough" guess for tlength when using a compressed
format.

sink-input: Kill passthrough streams if moving to an unsupported sink

This will eventually be replaced by a hook to let clients know that the
stream has moved so that they can gracefully reconnect and renegotiate a
supported format.

sink: Fix leak in pa_sink_check_formats()

We weren't freeing the sink formats idxset.

format: Avoid some code duplication

We frequently need to free an idxset containing pa_format_infos, so
define an internal free function that can be used directly with this
(instead of defining it once-per-file).

sink-input: Don't assert on bad formats

Handles bad format input more gracefully and returns an error instead.

sink-input: Return NOTSUPPORTED if format negotiation fails

This is easier for clients to grok than INVALID.

sink-input: Minor cleanups

Removes a couple of warnings and simplifies the assertion logic that
verifies format negotiation was successful.

tests: Add a trivial test for the extended API

This is just sync-playback.c modified to use the extended API. We need
something more sophisticated for properly testing the compressed
formats, but that is a non-trivial task in itself.

sink: Remove PASSTHROUGH flag

This removes the passthrough flag from sinks since we will drop
exclusively passthrough sinks in favour of providing a list of formats
supported by each sink. We can still determine whether a sink is in
passthrough mode by checking if any non-PCM streams are attached to it.

format: Add convenience API to check if a format is PCM or not

core: Add extended stream API to support compressed formats

This is the beginning of work to support compressed formats natively in
PulseAudio. This adds a pa_stream_new_extended() that takes a format
structure, sends it to the server (=> protocol extension) and has the
server negotiate with the appropropriate sink to figure out what format
it should use.

This is work in progress, and works only with PCM streams. Actual
compressed format support in some sink needs to be implemented, and
extensive testing is required.

More details on how this is supposed to work is available at:
http://pulseaudio.org/wiki/PassthroughSupport

sink: Extend API for compressed formats support

This adds a get_formats() vfunc for sinks to provide a list of formats
they can support. pa_sink_check_formats() can be used during or after
routing to determine what formats from a stream the sink can support.

format: Add some properties and internal API

The properties will be used by clients to set the sample format,
sampling rate, etc. The functions will be used internally.

core: Add a pa_format_info structure

This will be used to represent the format of data provided by the client
for both compressed and PCM formats in a new extended API.

sample: Use PA_SAMPLE_INVALID instead of numeric value

sink: Trivial typo fix

sink-input: Don't resample passthrough inputs

filter-apply: Mark modules as being autoloaded

(Based on Colin's review) We mark modules as being autoloaded so that
they can handle this as a special case if needed (which is required by
module-echo-cancel for now). This inverts how things were done and makes
using these modules manually less error-prone.

Make connect-stress test compile for win32

build-system: Small fixes

build-system: Rearrange database selection

build-system: Replace some more conditionals with AS_IF

build-system: Move dependency error messages to outer scope

build-system: Move AC_DEFINE to separate line with AS_IF

build-system: Use AS_IF macro for configure output

And add some HAVE_[feature] variables for clarity.

build-system: Simplify AC_ARG_ENABLE usage

Make use of the enable_[feature] variable automatically defined
by AC_ARG_ENABLE instead of defining our own variable.

combine: Rename module-combine to module-combine-sink.

This also includes a compatibility module that will ultimately be removed in
a future release.

bluetooth: Only close SCO if status has changed

In case the io thread is starting there is no point on closing SCO just
to resume it latter when the source/sink changes to opened state.

bluetooth: Fix using pointer-pointer when appending an array as variant

pa_dbus_append_basic_array_variant_dict_entry should take the direct
pointer to the array.

bluetooth: Fix not updating sample spec when using Media API

When using transport configured via Media API sample spec needs to be
updated since codec configuration may affect it when e.g. headset
configure a different frequency or number of channels from default.

x11: More XCB fixes.

Commit 65ef80b fixed building with xcb-util >= 0.3.8, but the reply is never
checked (possible SIGSEGV if the reply is NULL) nor freed (memory leak at each
call of the functions).

Also, remove include and dependencies on xcb-atom, as it was only meaningful
for xcb_atom_get() and STRING, and depend instead on xcb >= 1.6 for
XCB_ATOM_STRING.

dbus: Fix the order of freeing stuff when unloading module-dbus-protocol.

dbus: Fix connection idxset freeing when unloading the module.

If u->connections isn't empty when module-dbus-protocol is
unloaded, then connection_free() is called for the
remaining connections when the idxset is freed.
connection_free() tries to remove the connection from the
idxset, but that fails, because the item has already been
removed from the idxset in this scenario.

The problem is solved by not trying to remove the connection
from the idxset in connection_free(). Instead, whoever wants
to delete connections, has to remove the connection from the
idxset in addition to calling connection_free().

match: Support for both merging and replacing proplist updates.

This patch adds a new update mode specifier that can be optionally
given in match rules after the regexp. Property list updates triggered
by the rule will honour the given mode. The two allowed modes are 'merge'
and 'replace', corresponding to PA_UPDATE_MERGE and PA_UPDATE_REPLACE
respectively. If omitted, the mode defaults to PA_UPDATE_MERGE, ie. to
the original behavior.

For example, to force 'media.role' to be overwritten with 'bar' for
streams matching foo you can use an entry like this:

foo replace "bar"

This will really overwrite media.role to bar even if it has already been
set to something else by the application.

Thanks to Krisztian Litkey for the original patch and the description
above. In addition to implementing the new feature, this patch fixes
a number of bugs in the parsing code.

core: Drop empty gaps in the memblockq when playing data from it.

It's possible that the memblockq of a sink input is rewound to a negative read
index if the sink input is moved between sinks shortly after its creation. When
this happens, pa_memblockq_peek() returns a memchunk whose 'memblock' field is
NULL and whose 'length' field indicates the length of the gap caused by the
negative read index. This will trigger an assert in play-memblockq.c.

If the memblockq had a silence memchunk, pa_memblockq_peek() would return
silence for the duration of the gap and the assert would be avoided. However,
this approach would prevent the sink input from being drained and is thus not
possible. Instead, we handle the aforementioned situation by dropping the gap
indicated by the 'length' field of the memchunk and by peeking the actual data
that comes after the gap.

This scenario seems to be quite rare in everyday use, but it causes a severe
bug in the handheld world. The assert can be triggered e.g. by loading two null
sinks, playing a sample from the cache to one of them and then moving the
created sink input between the two sinks. The rewinds done by the null sinks
seem to be quite long (I don't know if this is normal behaviour or something
fishy in module-null-sink).

See also:

    6bd34156b130c07b130de10111a12ef6dab18b52
    virtual-sink: Fix a crash when moving the sink to a new master right after setup.

    https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009105.html

Reproduce:

This problem can be reproduced with the following script:

SAMPLE_PATH="/usr/share/sounds/alsa/"
SAMPLE="Front_Left"

pactl remove-sample $SAMPLE 2> /dev/null
pactl upload-sample $SAMPLE_PATH$SAMPLE.wav

mod1=`pactl load-module module-null-sink sink_name=null1`
mod2=`pactl load-module module-null-sink sink_name=null2`

pactl play-sample $SAMPLE null1

input=`pactl list | grep "Sink Input #" | tail -n 1 | cut -d# -f2`

echo "Sample $SAMPLE playing as Sink Input #$input"

pactl move-sink-input $input null2
pactl move-sink-input $input null1

pactl unload-module $mod1
pactl unload-module $mod2

stream-restore: Enable database dumping if DEBUG_VOLUME is defined.

module-null-source: New null-source module

x11: Fix build errors with newest xcb-util.

The xcb_atom_get functions were removed from xcb-util. Changed these to
xcb_intern_atom/xcb_intern_atom_reply. Also, STRING is now
XCB_ATOM_STRING.