format: Update map-file

Adding property getters manually for now.

format: Add more property getters

This adds integer range/array and string array property getters to the
pa_format_info API. Corresponding tests added as well to ensure the code
is valgrind-clean.

The corresponding functions are added to map-file manually for now.

format: Expose pa_format_info<->pa_sample_spec conversion functions

These utility functions could be handy to clients.
pa_format_info_to_sample_spec_fake() isn't made public, but the return
value is changed to keep in sync with pa_format_info_to_sample_spec().

format: Don't assert on errors in getters

This makes handling errors in getter functions more graceful, rather
than triggering warnings/asserts. Better to be less trigger-happy about
these things since this is now public-facing API.

format: Export pa_format_info int and string property getters

We currently only have setters and clients need to be able to query
these values as well. The return types for these functions needed to be
changed to int since this is public API now.

bluetooth: Run update-sbc

This is primarily for the bluez commit 03bb9d3 by Siamashka Siamashka
which fixes a compilation error with gcc 4.7

alsa: add DTS profile

Hi. Could you please apply the attached trivial patch so that I could drop
the corresponding instructions from dcaenc's README file in the future? It
adds a profile for on-the-fly DTS encoding, similar to the existing AC3
profile.

--
Alexander E. Patrakov
>From 22310a1c28385acc7ce883e020b9eb2e5b0813b7 Mon Sep 17 00:00:00 2001
From: "Alexander E. Patrakov" <patrakov@gmail.com>
Date: Sun, 12 Feb 2012 17:19:48 +0600
Subject: [PATCH] alsa: add DTS profile

This requires dcaenc from http://aepatrakov.narod.ru/dcaenc/

bluetooth: Fix calling many times Audio.GetProperties for the same device

Audio.GetProperties is been called for as many times as there are UUIDs
instead of just once when the UUIDs are discovered.

build: Fix out-of-tree build

Previous commits broke the out-of-tree build on padsp.

.gitignore: Add padsp to gitignore

build: Force order of library installation

libtools causing relinking on installation, to make this succeed
libpulsecommon needs to be installed before the other libraries and the
padsp libraries needs to be installed afterwards.

Unfortunately autotools doesn't consider dependencies when running the
install target, thus we have to enforce the ordering ourselves

padsp: Move the padsp helper lib into a private library

libpulsedsp is only used as a LD_PRELOAD library
for the padsp wrapper. So no need to have it in
a public library

build: Move libpulsecommon into $pkglib

libpulsecommon is a private library only for use within pulseaudio, so
lets move it into a private directory

[ed: from discussion on IRC, while it looks redundant to have modules in
lib/pulse-$VER and private libraries in $pkglibdir, this is actually
desirable. For example, a multi-architecture libpulse would have a
$pkgdir per architecture (/usr/lib/<arch>/pulseaudio) whereas you'd
still want the server-specific modules in /usr/lib/pulse...) -- Arun]

tunnel: fixup create_record_stream

The tunnel source has been broken since protocol v22 (PA 1.0),
and connecting fails with a protocol error. Fix.

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

PROTOCOL: Fix documentation for version 22

Fix the documentation: the protocol file was not reflecting the code
properly for version 22.

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

alsa-mixer: Don't use dangling pointers as port hashmap keys

If alsa settings/options were used, the string to construct the
name was freed, leading to dangling pointers to strings inside
the hashmap.

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

pulse: Fix old bug in stream_free

Signed-off-by: Maarten Lankhorst<m.b.lankhorst@gmail.com>

cli: Ensure source output volumes are printed via cli interface (pacmd ls)

pacmd: Fix compiler warning

We were comparing an int with a size_t.

man: Document the local-server-type daemon.conf option.

virtual-sink: Remove irrelevant comment.

The comment is inherited from module-ladspa-sink. It
doesn't make sense in module-virtual-sink.

core: svolume tests should generate realistic random volume data

assuming RAND_MAX is around 1<<31, rand() >> 1 generates large numbers as
random volume data; these likely causes saturated sample values after
applying the volume function -- not a good test

core: whitespace typo

alsa: fix comment

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