sample-util: Fix "Darth Vader" panning bug

For muted channels, we forgot to increment a pointer, so if one
channel was muted but not the other, sound became distorted in a
Darth Vader like way. To test the difference, start two input
streams and pan one of them hard left (or right).

And hey, if you didn't think it sounded like Darth Vader, it's
your imagination that's broken, not mine! ;-)

BugLink: https://bugs.launchpad.net/bugs/928757
Signed-off-by: David Henningsson <david.henningsson@canonical.com>

sink-input,source-output: Handle devices going away in unlink hooks

If a *_UNLINK_POST hook causes a sink-input/source-output's sink/source
to go away, the subsequent attempt to update the sink/source status will
cause an assert. We deal with this by checking the sink/source status
before trying to update it.

loopback: Trivial whitespace fix

loopback: Fix crash when moving sink-input fails

format: Add "since 1.0" documentation tags where they were missing

alsa-mixer: Take override-maps into account in subset elimination

The practical problem is that some users were left with only one
"LFE on Mono" port, as analog-output was considered a subset of
analog-output-lfe-on-mono. Which was not what they wanted.

BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=40910
BugLink: https://bugs.launchpad.net/bugs/922656
Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa-mixer: Make sure unsupported paths are removed after probing

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

loopback: Fix crashes

Flush the message queue before tearing down, and dest==NULL is valid in case moving failed.

With this my module-loopback finally no longer causes frequent crashes.

i18n: Run make update-po

i18n: Do not translate strings that cannot have any sensible translations.

i18n: Fudge translations after previous commit to avoid mixing English/localized phrases.

The translations are now obviously outdated and wrong, but as the
core meaning has not changed, this is the lesser of two evils.

alsa: Improve "well known descriptions" for ports

 * If we don't have "Digital Speakers", we should say "Speakers"
   instead of "Analog Speakers", and similar for other ports.
 * Change "IEC958" to "S/PDIF" (more well known name)
 * Add new ports and mappings for HDMI
 * Change "Internal" to "Built-in" for the card name

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa-mixer: Allow speaker port to control "Front Speaker"

"Front Speaker", "Surround Speaker" seems to be a common enough name
to make it into alsa-utils, so we should probably care about it as
well. In this case, there was a macbook pro whose speakers didn't work
without these controls.

BugLink: http://bugs.launchpad.net/bugs/551441
Reported-by: Jeroen T. Vermeulen <jtv@canonical.com>
Signed-off-by: David Henningsson <david.henningsson@canonical.com>

fix the ever-popular 'the the' typo

some are comments, some are user-visible doxygen text and documentation

manpage: document --log-meta, --log-time, log-backtrace

manpage: document --log-target=file:PATH

stream-restore: Don't verify entry validity needlessly.

clean_up_db() makes sure that all entries in the database
are valid.

stream-restore: Clean up the database at startup.

Fixes a crash: https://bugs.freedesktop.org/show_bug.cgi?id=44522

pulse: Document general error handling.

bluetooth: Change function name add_matches to update_matches.

A function that is used for removing matches should not be
called add_matches.

bluetooth: Remove the right match in the proximity module.

The original intention of this code was probably that if
adding filter1 succeeded but adding filter2 failed, then
filter1 should be removed so that either both or none of the
filters get added.

dbus: Give NULL as the error parameter to dbus_bus_remove_match().

We didn't do anything anyway in case of failures. When we
give NULL as the error, dbus_bus_remove_match() can act
asynchronously, so it becomes faster. Also, the bus daemon
can avoid sending any replies, which reduces the amount of
traffic.

Remove pa_prioq priority queue implementation

alsa: Fix log message "synchronous volume" -> "deferred volume"

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa-mixer: Fix mixer path for AC3 profiles

The recent change to turn off the IEC958 element for analog paths
exposed a bug in AC3 profiles. These were inheriting the analog output
path instead of explicitly selecting the iec958 path.

Thanks to David Henningsson for pointing this out.

bluetooth: When receiving D-Bus errors, print also the error message.

dbus: New helper function: pa_dbus_get_error_message().

pulse: Turn the anonymous error code enum into pa_error_code_t.

The purpose is to make it possible to link to the
enumeration in doxygen.

stream: Clarify the sign of error return codes

The returned error values are all negative, but the documentation did
not explicitly mention this. Thanks to Vincent Penquerc'h for pointing
this out.

alsa-mixer: Turn off the IEC958 element for analog outputs

This is needed for the Creative Audigy CA0106 to work. Also makes sure
that the LED for optical out is shut down in analog modes on MacBooks
(these share a port for analog and digital output).

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=44741

sconv: Fix generation of floats in SSE test code

the random floats x should be zero-mean with -(1+eps)<x<(1+eps) for some
small epsilon; previously only negative x were generated

core: Fix log message about ARM feature detection

core: Fix return of pa_cpu_init_arm()

alsa: Mention correct ALSA function in debug log

orc: Another trivial documentation fix

orc: Trivial documentation typo fix

alsa-mixer: Refactor pa_alsa_profile_set_probe().

Increase readability of the pa_alsa_profile_set_probe function of
alsa-mixer.c

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa-mixer: When selecting an input, turn off boosts of other inputs

Just like we turn off the "Front Mic" element when we select "Rear Mic",
we should also turn off the "Front Mic Boost" element. And the same for
the other inputs.

Reported-by: Len Owens <len@ovenwerks.net>
Signed-off-by: David Henningsson <david.henningsson@canonical.com>

daemon: Drop --module-idle-time from docs

This option does not exist any more.

daemon: Fix *-idle-time arguments

--exit-idle-time and --scache-idle-time were marked as having an
optional argument when the argument is actually mandatory. This causes a
crash when using this argument.

Thanks to Matthijs Kooijman (blathijs on IRC) for pointing this out.

pacmd: dynamically allocate ibuf and obuf

Use pa_pipe_buf to determine the minimum size for ibuf and obuf, taking into
account the two descriptors that use each of them.

See bug #42715

mutex: handle gracefully if a PTHREAD_PRIO_INHERIT protocol cannot be set

This adds an additional check for unavailable PTHREAD_PRIO_INHERIT to the
fallback work done in ca717643ee768307475fc36ea29d920a13db0a8e

See bug #42715

module-rtp-recv: fail when SO_TIMESTAMP is not defined

SO_TIMESTAMP is not POSIX and not available in any platform, so just fail
if the current platform does not have it.

See bug #42715

rtp: use the right type when checking cmsg_type

Use SCM_* instead of SO_* when checking the type of each cmsghdr.

See bug #42715

pipe: use pa_pipe_buf instead of the macro PIPE_BUF

This helps when porting to platforms that lack PIPE_BUF.

See bug #42715

doc: Clarify pa_stream_get_latency() return value

Clarifies that the latency is returned via an inout parameter and the
return value is an error code or 0 on success.

build-sys: Make esound bits optional

This is really quite irrelevant on most modern systems today, and more
so for embedded systems.

core: comment typo

core: fix typo in logging

core: sample_spec.rate is in Hz, not kHz; change logging output

core: fix potential memory leak

build-sys: Fix po/ build with --disable-nls

We can't call IT_PROG_INTLTOOL because it requires intltool even with
--disable-nls, but it provides USE_NLS which is required for the po/
build. This might take a while to get fixed upstream, so working around
this in our build system for now.

build-sys: Fix map-file check

configure doesn't provide an abs_top_srcdir early enough, so we just use
srcdir for that. There was also a problem with the path we were setting
in VERSIONING_LDFLAGS that is fixed now.

svolume_{mmx, sse}, sconv_sse: Fix compilation errors with X32 toolchain

From d8b81d5393df36085009bf9f69d41fa85e2ae58a Mon Sep 17 00:00:00 2001
From: Nitin A Kamble <nitin.a.kamble@intel.com>
Date: Sat, 10 Dec 2011 09:09:06 +0100

Make assembly syntax compatible to the X32 toolchain and fix the
following kind of compilations errors with X32 gcc.

| pulsecore/svolume_mmx.c: Assembler messages:
| pulsecore/svolume_mmx.c:107: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:135: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:161: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:162: Error: `8(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:180: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:210: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:244: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:245: Error: `8(%esi,%rdi,4)' is not a valid base/index expression
| make[3]: *** [libpulsecore_1.1_la-svolume_mmx.lo] Error 1

Originally these assembly lines were written for x86_64 ABI, now they
are also compatible with X32 ABI [3][4].

The patch was submitted to the OpenEmbedded-Core list [1][2].

[1] http://lists.linuxtogo.org/pipermail/openembedded-core/2011-December/014189.html
[2] http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=nitin/x32&id=2d8eec54f755c51f2eff600390f5a4b3cc2a7662
[3] https://wiki.yoctoproject.org/wiki/X32_abi
[4] http://en.wikipedia.org/wiki/X32_ABI

x11: Fix build without NLS support

tests: Fix resampler-test compilation without NLS support

echo-cancel: Better handling of error conditions in test

echo-cancel: Begin log message with uppercase letter

echo-cancel: Set file mode to binary in test code

protocol-native: Fix 'auth-group-enabled' modarg

This is actually implemented in module-protocol-stub as
'auth-group-enable'. An unfortunate typo because the other argument is
spelt as 'enabled', but it's better to be slightly inconsistent than to
change the public interface.

resampler: Move some peak resampler asserts around

This moves a couple of asserts from peak_resample() to peaks_init()
since they're resampler parameters that shouldn't change after
initialisation.

resampler: Remove invalid channel asserts in peak and trivial

The incoming channel count will be fixed up by the remapping code before
the resampler is invoked.

utils: Typo fixes in qpaeq

Pointed out by John Frankish <j-frankish@slb.com> in:

https://bugs.freedesktop.org/show_bug.cgi?id=43733

echo-cance: Make Adrian canceller optional

makes the Adrian echo canceller implementation optional at compile time

this patch supersedes an earlier patch proposal and addresses the following
comments:
* separate patch from speex dependency rework (Arun)
* check that at least one EC implementation is available (Arun)
* properly align yes/no in configure summary for Adrian (Frederic)

build-sys: Make speex library optional

make speex library dependency optional, this affects the resampler
and the echo canceller module

this patch supersedes an earlier patch proposal and addresses the following
comments:
* fix order of pa_echo_canceller_method_t enum and ec_table (Frederic)
* the default resampler is speex if available as before, otherwise ffmpeg (Arun)
* does not touch the Adrian EC implementation (see separate patch) (Arun)

utils: Fixes for building without NLS

iochannel: Handle missing un.h correctly

mime: Move assert to correct position

build-sys: Use absolute path for map-file while linking

This makes the link process more robust in some stranger configurations.

build-sys: Fix building without NLS

stream: Fix typos and formatting in stream.h doxygen documentation

mainloop: Fix typos and rewording of thread-mainloop.h doxygen documentation

simple: Fix typos in simple.h doxygen documentation

stream: Fix 'e g' as 'e.g.\ '

error: Fix spelling of 'initialization' in errortab for PA_ERR_MODINITFAILED

Also update the related translations.

proplist: Consistently use parameter name p for a pa_proplist* in prototype

utf8: Fix doxygen file description for utf8.h

proplist: Fix typos in doxygen documentation

proplist: Add doxygen file description for proplist.h

introspect: Fix unresolved doxygen link pa_port_available_t

format: Add description for format.h

format: Add periods to fix brief documentation for pa_format_info_is_pcm()

format: Fix unknown doxygen command \second

def: Document that pa_buffer_attr.maxlength is in bytes.

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=42919

stream: Fix comments

pstream: Fix spelling of 'receive'.

These functions are not exported in the map file and thus do
not form part of the API and can be corrected without any problem.

alsa-mixer: Remove unused pa_alsa_path_set_probe() declaration.

device-port: Add a property list to ports.

They're not used for anything yet, but hopefully soon they'll be.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

pactl: Show port availability

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Fixup a few things in the new path probing

 * If mapping_probe_paths() fails to open the mixer, all paths are now
removed from the mapping's path sets.
 * pa_path_set.probed isn't really used for anything (removed).
 * If profile probing is configured to be skipped, mapping_paths_probe()
should still be called.

Thanks to Tanu for spotting.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa: add card ports and path probe cache

To be able to add ports to all profiles, we need to probe all
profiles at startup. To speed this up, we now have a cache of
probes paths which is owned by the profile set. Since paths
are now owned by the profile set, the path set must now have
a hashmap of paths instead of a linked list.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa: Ignore the virtual "thinkpad-acpi" card

The thinkpad ACPI driver sometimes creates a virtual sound card,
which at best exposes a volume control. Save some startup time, and
unnecessary error messages in the log, by ignoring it.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

echo-cancel: Fix memory leak in test program

memory for pa_echo_canceller is allocated in init_common() a second time,
hence the allocated memory in main() is leaked

resampler: Some minor tweaks

that were forgotten in the previous peaks resampler commits.

echo-cancel: Fix warning for undefined HAVE_WEBRTC

tests: Don't link tests that only use public api to libpulsecommon

resamplers: Improve performance of peaks resampler

This is mainly achieved by special-casing the common 1ch float case,
which is used by applications such as pavucontrol.

resamplers: Rework the peaks resampler

The algorithm had been implemented the same way as the trivial resampler.  But
an important difference between the two is that the trivial resampler can write
an output as soon as the first corresponding input sample is seen, whereas the
peaks resampler must have read all input samples before writing an output
sample.

With this rework, the peaks resampler now outputs samples correctly when the
input data is spanning multiple memblocks.

shm: Use a goto rather than early return for consistency.

This is functionally the same but is easier to understand.

win32: Avoid some compiler warnings when cross-compiling for mingw32

Autoconf documentation says that AC_FUNC_SELECT_ARGTYPES shouldn't be used anyway.