alsa: Remove pointless initialization.

The value stored in msg is never read. Found by the clang analyzer.

alsa_mix: Allow non-positive mixer values.

For the rpi the PCM range is from -10239 to 400. The math in
->set_mixer works fine for negative values.

alsa: New writer option: --buffer-time.

This removes the old way of setting of private_alsa_write_data->buffer_time

Currently we set the buffer time to the minimum of the maximum
buffer time and 500ms. This results in different values for different
hardware. In view of the introduction of the sync filter it seems to
be desirable to have more control over this value.

This patch adds the --buffer-time config option to the alsa
writer which allows to specify a suitable value, the default being
170ms. The unit of private_alsa_write_data->buffer_time is changed
from microseconds to milliseconds as this avoids to divide by 1000
in ->pre_select().

alsa: Work around poll fd problems.

In its ->pre_select method the alsa writer adds the alsa poll file
descriptor to the set of fds to be monitored during the next select
call. When this fd is ready for I/O, select() returns and ->post_select
is called. Currently ->post_select checks if the alsa poll fd is set
and returns early if it is not.

The problem is that on some hardware/kernel/alsa-lib combinations,
the poll fd is never marked as ready for I/O. This happens at least
on the raspberry pi with linux-3.10.10 and alsa-lib-1.0.27. The result
is no audio at all.

We address this problem in two ways:

* In alsa_init() we remember the buffer duration for later use in
->pre_select where we ask the scheduler to impose half of the duration
as an upper bound on the select timeout.

* In ->post_select, if there is input data available, we don't
check any more whether the poll fd is ready but try to write
unconditionally. On EAGAIN, we read from the poll fd to discard any
pending events.

alsa: Set period time.

This is probably not necessary, but since aplay from the alsa-utils
package sets it in the same way, it can't hurt to do this.

alsa: Also dump hardware params.

Since this makes the alsa output even more verbose, let's downgrade
the loglevel of config dump messages to DEBUG.

alsa: Improve help text of --device.

The new text gives the user a hint how to find out about the devices
ALSA knows about.

alsa: Improve documentation of struct private_alsa_write_data.

Remove the stale reference to writer node groups and explain how
sample_rate and channel count are determined.

alsa: Don't set t->error in ->pre_select().

->pre_select() methods should never fail, and tasks should not
set this variable at all. It is ignored anyway when set there.

Merge branch 't/cmd_improvements'

Cooking since 2014-01-16.

* t/cmd_improvements:
  afs.cmd: Strip redundant "sort" from ls help output.
  audiod.cmd: Improve grab client help text.
  afs.cmd: Improve second level indenting.
  afd.cmd: Fix a trivial whitespace issue.

Merge branch 't/sync'

Cooking since 2014-01-14.

* t/sync:
  The sync filter.
  net: Let maksock() callers perform flowopt cleanup.
  net: makesock_addrinfo(): Make socketfd local to the loop.
  net: Let makesock() continue on setsockopt() failure.
  net: Reduce indentation level in makesock_addrinfo().
  net: Further simplify makesock_addrinfo().
  net: Replace the double loop of lookup_address() by a single loop.
  net: makesock(): Combine code for passive sockets.
  net: Change makesock_addrinfo() to set given flowopts before SO_REUSEADDR.
  net: Clarify code flow of makesock_addrinfo().
  net: Remove unnecessary condition in makesock_addrinfo().
  net: Remove pointless initialization in makesock_addrinfo().
  net: Kill dead code in makesock_addrinfo().
  net: Remove networking headers from para.h.
  net: Improve error handling of makesock_addrinfo().
  net: Split makesock(), part 2: Introduce makesock_addrinfo().
  net: Split makesock(), part 1: Introduce lookup_address().
  net: Simplify makesock().
  net: Fix parse_url().
  net: Correct \return text of parse_url().

Merge branch 'refs/heads/t/clang_warning_fixes'

Was cooking since 2014-01-14.

1b9e28 command.c: Silence clang warning.
994e72 udp_recv: Fix clang warning.
c853e7 aac_afh: Fix clang warning.

afs.cmd: Strip redundant "sort" from ls help output.

The sentences before this table make it clear that these options
change the sort order.

audiod.cmd: Improve grab client help text.

The shortened text does not mention file descriptors any more, it
should be easier to understand.

Also reformat the text slightly.

afs.cmd: Improve second level indenting.

Currently we have an empty line between each item, mainly to make sure
that man pages including the html version look nice. However, indenting
by spaces and dropping the empty lines looks better in the plaintext
version (help output) while still keeping help2man and man2html happy.

afd.cmd: Fix a trivial whitespace issue.

The sync filter.

This adds a new filter for synchronization between clients. It works
by sending an UDP packet to other clients ("buddies").

To reduce latency, address resolution is only performed once on
startup.  Hence lookup_address() and makesock_addrinfo() of net.c
are made public.

This commit introduces new public function sockaddr_equal() in
net.c which compares two IPv4 or IPv6 addresses. It is used in
sync_find_buddy().

net: Let maksock() callers perform flowopt cleanup.

Currently the flowopt user allocates the flowopt structure but
cleanup is performed in makesock(). This commit makes the API
symmetric by dropping the cleanup part in makesock() and exporting
flowopt_cleanup(). Now callers have to perform both the allocation
and the cleanup.

The single user (ddcp_recv) is adjusted accordingly.

net: makesock_addrinfo(): Make socketfd local to the loop.

And rename it to fd, as there is no other file descriptor in this
function.

net: Let makesock() continue on setsockopt() failure.

If the setsockopt() call succeeds but the following bind() fails, we
happily continue to process the address info structure. It seems wrong
to abort the loop if setsockopt() could not enable SO_REUSEADDR. Just
continue in this case.

net: Reduce indentation level in makesock_addrinfo().

Handle the active socket case of makesock_addrinfo() first, i.e.
transform

if (a)
lots_of_stuff;
else
short;
into

if (!a) {
short;
continue;
}
lots_of_stuff;

net: Further simplify makesock_addrinfo().

Instead of breaking out of the loop, we may as well return the
appropriate value directly.

net: Replace the double loop of lookup_address() by a single loop.

net: makesock(): Combine code for passive sockets.

Make use of  the fact that

if (a && b)
foo;
if (a)
bar;
else
baz;

is equivalent to

if (a) {
if (b)
foo;
bar;
} else
baz;

but the second form is easier to read, IMHO.

net: Change makesock_addrinfo() to set given flowopts before SO_REUSEADDR.

Should not matter but allows to combine the code that follows.

net: Clarify code flow of makesock_addrinfo().

src != NULL is equivalent to passive, and dst != NULL equivalent
to !passive.

net: Remove unnecessary condition in makesock_addrinfo().

This code is executed only if src != NULL, which implies dst == NULL.

net: Remove pointless initialization in makesock_addrinfo().

net: Kill dead code in makesock_addrinfo().

Either src or dst is NULL, hence the condition is never true.

net: Remove networking headers from para.h.

Most files don't need these, so include all headers related to
networking in each individual file.

net: Improve error handling of makesock_addrinfo().

Rename rc to ret, and always set this variable so that an appropriate
error code will be returned in all cases.

command.c: Silence clang warning.

command.c:562:32: warning: Null pointer passed as an argument to a 'nonnull' parameter
        return send_sb(&cc->scc, msg, strlen(msg), SBD_OUTPUT, false);

Harmless, the newly added assert() makes scan-build STFU.

net: Split makesock(), part 2: Introduce makesock_addrinfo().

This extracts the addressinfo part of makesock() into the new
makesock_addrinfo().  Hence makesock() becomes a simple function
which first calls lookup_address(), then makesock_addrinfo().

The two helper functions lookup_address() and makesock_addrinfo()
can be made public once there is a need for them.

net: Split makesock(), part 1: Introduce lookup_address().

The lookup part is relatively independent and might be useful on its
own, so create lookup_address().

net: Simplify makesock().

l3type is always AF_UNSPEC in this function.

net: Fix parse_url().

Does not correctly parse host, if given array has no extra
space.

net: Correct \return text of parse_url().

The parameter is called port rather than portnum. Reformat
the paragraph and fix the font type of NULL while we are at
it.

reset version to 'git'

paraslash 0.5.1

Merge branch 't/build_system_overhaul'

This topic was quite some PITA to maintain since its commits heavily
conflict with the changes of all other topic branches which touch
the build system. Cooking since 2013-12-01 and tested on FreeBSD,
NetBSD and on different Linux systems.

abb080 Rebuild error2.h on Makefile changes.
71b616 manual: Move crypt library to the list of optional packages.
185c67 build: Don't insist on curses for CLI support.
474d4d build: Rewrite objlist_to_errlist().
1453e8 build: Reorder Makefile.real.
7fb39c build: Split Makefile.in.
d0ddd2 build: Reorder variables in Makefile.in.
61cc38 build: Add Make variable for clock_gettime_ldflags.
976651 build: Add Make variable for PACKAGE_TARNAME.
19b0d3 build: Introduce Make variables for objects.
fe7666 build: Combine link rules.
2440c4 build: Introduce $prefixed_executables.
2f113b build: Combine rules for object files.
761d9b build: Define Make variables for cppflags.
386572 build: Remove two unused variables from Makefile.in.
40439a build: Collect audio format handlers in afh section.
e926e8 build: Move relevant parts to filter section.
b399d6 build: Move relevant parts to gui section.
1781cc build: Move relevant parts to fade section.
0f4790 build: Move relevant parts to audiod section.
4e9788 build: Move relevant parts to client section.
0c8626 build: Move relevant parts to server section.
755904 build: Create section for para_audioc.
d952d6 build: Create section for para_write.
9734df build: Create section for para_play.
14dee3 build: Create section for para_afh.
b1abc2 build: Create section for para_recv.
bfd5b5 build: Create section for para_filter.
9139d9 build: ldflags conversion: coreaudio.
30f966 build: ldflags conversion: libcurses.
15880b build: ldflags conversion: libnsl.
516421 build: ldflags conversion: libsocket.
ce8273 build: ldflags conversion: libgcrypt.
ad3573 build: ldflags conversion: libssl.
52ac85 build: ldflags conversion: libosl.
37083e build: ldflags conversion: libsamplerate.
7de07a build: ldflags conversion: libreadline.
0e9f1c build: ldflags conversion: libao.
ecac89 build: ldflags conversion: alsa.
6c632d build: ldflags conversion: liboss.
e74262 build: ldflags conversion: libflac.
648197 build: ldflags conversion: libmad.
91ed86 build: ldflags conversion: libfaad.
8ee713 build: ldflags/cppflags conversion: libopus.
b6c705 build: cppflags/ldflags conversion: libspeex.
7aa414 build: ldflags conversion: libvorbis.
80a639 build: ldflags conversion: libogg.
c79cdc build: cppflags/ldflags conversion: libid3tag
a02396 build: Remove duplicates from $all_objs.
d2fc4d build: Remove inactive DEBUG_CPPFLAGS.
e8f310 build: Replace error2.pl by error2.c.
06e33e build: Fix filter output of configure summary.
994889 build: Convert remaining object lists to one object per line format.
782e47 build: Separate curses detection from gui configuration.
fbe4b0 build: Move down fade section.
003815 build: Add object "cmdline" suffix only once.
3097e5 INSTALL: Add an example for cross-compiling.
e8b555 build: Fix make install for cross builds.
aab667 build: Generate man pages directly from ggo files.
9a8a15 build: Remove fnmatch, malloc, realloc tests.
2dcdd9 build: Make crypto optional.
f23353 build: Make audiod optional.
98b801 build: Build para_client only if crypto lib was found.
17ed11 build: Create a separate section for para_server.
02c1e0 build: Compute $all_errlist_objs.
4ec04c build: Replace $extras by per-executable variable.
39707e build: Remove two dead recipes from Makefile.in.
9fc99c build: Remove check for buggy snprintf().

Merge branch 't/opusdec-latency'

A single patch which was cooking since 2013-11-17 with no problems.

14c16b opusdec: Latency improvements.

Rebuild error2.h on Makefile changes.

Re-running configure without a subsequent make clean currently fails if
the set of objects has changed since the last run. This happens for
example if a different cryptolib was requested with --enable-cryptolib,
or if different libraries were detected by configure.

The problem is that error2.h is not updated according to the new
set of objects. Letting error2.h depend on Makefile should fix it,
because configure creates Makefile, so error2.h will be recreated
after reconfiguring.

Merge branch 't/aft_cleanups'

Was cooking since 2013-11-10.

fb4022 aft: Make get_afsi_of_path() static.
89d124 aft: Make load_afsi() and safe_afsi() static.
5a63e0 aft: Make get_afsi_object_of_row() static.

udp_recv: Fix clang warning.

clang does not seem to like adding to a string literal:

udp_recv.c:51:45: warning: adding 'size_t' (aka 'unsigned long') to a string does not append to the string [-Wstring-plus-int]
if (memcmp(iov[1].iov_base, FEC_EOF_PACKET + iov[0].iov_len,
    ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
udp_recv.c:51:45: note: use array indexing to silence this warning

That's kind of silly, but if this simple change makes clang happy,
so be it.

aac_afh: Fix clang warning.

Although gcc seems to have no problem with it, clang complains about
illegal characters in string literals of aac_afh.c:

aac_afh.c:114:25: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
if (!atom_cmp(type1, "<A9>ART"))
      ^~~~
aac_afh.c:116:30: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
else if (!atom_cmp(type1, "<A9>alb"))
   ^~~~
aac_afh.c:118:30: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
else if (!atom_cmp(type1, "<A9>nam"))
   ^~~~
aac_afh.c:120:30: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
else if (!atom_cmp(type1, "<A9>cmt"))
   ^~~~
aac_afh.c:122:30: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
else if (!atom_cmp(type1, "<A9>day"))

This patch encodes the offending value as "\xa9" to make clang happy,

Merge branch 't/flacdec_fixes'

Cooking since 2013-10-27.

6da1ca flacdec: Fix DECODER_ABORTED handling.
76a2c8 flacdec: Avoid busy loop.
a1beef flacdec: Fix mono output.
8633a2 flacdec: Avoid declaration after statement.
464648 flacdec: Improve documentation of unconsumed.
c2d841 fd.c: Fix bad English in comment.

manual: Move crypt library to the list of optional packages.

Commit 2dcdd92c made crypto an optional feature. So this item should
go to the list of optional packages.

build: Don't insist on curses for CLI support.

On systems without curses headers/libs, support for interactive
cli tools is deactivated and a message is shown which claims that
readline depends on the curses library. This is not true: readline
uses the termcap functions, but does not link with the termcap or
curses library itself.  Applications must choose an appropriate
library that provides these functions.

This change should enable readline support on systems that have
libtermcap but no curses library.

build: Rewrite objlist_to_errlist().

This function of configure.ac was defined as a single line with 225
characters. Splitting it into three smaller parts makes it readable
again.

build: Reorder Makefile.real.

Trivial cleanup. Only code move, no actual changes.

build: Split Makefile.in.

This trims down Makefile.in to contain only the initializations of
Make variables with their autoconf counterpart. The new Makefile.real
contains the second part, including all recipes.

With this commit, Makefile.in needs to be changed only when
configure.ac introduces new variables. In this case a full
autoconf, configure, make cycle is necessary anyway. However, if
only Makefile.real changes, a simple make run should be enough to
update everything.

build: Reorder variables in Makefile.in.

Move down derived variables so that the top of Makefile.in consists
only of definitions of Make variables from autoconf variables.

build: Add Make variable for clock_gettime_ldflags.

Another autoconf variable reference bites the dust.

build: Add Make variable for PACKAGE_TARNAME.

This gets rid of two more references to autoconf variables.

build: Introduce Make variables for objects.

The definition of the various foo_objs variables still refers to the
autoconf variables. Having a make variable for each set of object is
another step towards the separation of autoconf and make.

build: Combine link rules.

Due to the cleanups in the previous patches, all rules for linking
have become identical. Hence we may replace all of them by a single
one.

build: Introduce $prefixed_executables.

Currently $executables contains the full basename of all
executables (para_server, para_client, ...), while the autoconf
variable @executables@ is used for the variants without the "para_"
prefix. This is rather confusing, so introduce $prefixed_executables
and stop using the @executables@ variant in Make recipes.

build: Combine rules for object files.

Currently we have many similar recipes for compiling object files from
.c files that need non-standard CPPFLAGS. All rules are identical
except that different additional options to the $(CC) command are
required to build the object file. These additional options are
provided through various $(foo_cppflags) variables.

This commit avoids this duplication by adding the necessary flags to
CPPFLAGS instead. This way we can use the same rule for all object
files.

build: Define Make variables for cppflags.

As a first step towards separating autoconf output variables and Make
recipes, this commit defines Make variables for all cppflags at the
top of Makefile.in and avoids @variable@ constructs in Make rules.

As an additional benefit, this improves readability of the generated
Makefile.

build: Remove two unused variables from Makefile.in.

PKGDATADIR and PACKAGE_STRING are defined but never used. Remove them.

build: Collect audio format handlers in afh section.

The afh section is the natural place to define this variable.

build: Move relevant parts to filter section.

This code deals with variables and defines for the various filters,
so it should go to the filter section.

build: Move relevant parts to gui section.

The definition of gui_objs and INIT_GUI_ERRLISTS belongs to the
gui section so move it there.

build: Move relevant parts to fade section.

configure.ac already has a section for para_fade, but the construction
of $fade_errlist_objs was still spread out over the whole file. Move
everything to the existing section and remove @fade_ldflags@.

This also kills the $mixer_summary variable, which was not very useful.

build: Move relevant parts to audiod section.

All of this should be in the audiod section, so move it there. Due
to the cleanups of the previous patches the @audiod_ldflags@ output
variable contained only -lm. Get rid of it by adding -lm to LDFLAGS.

build: Move relevant parts to client section.

These parts belong to the client section at the bottom of configure.ac.
This also gets rid of @client_ldflags@.

build: Move relevant parts to server section.

Currently $server_errlist_objs is updated whenever a feature that
affects para_server was detected. This mix of feature detection and
build options is hard to maintain and error-prone, so let's move the
definition of $server_errlist_objs to the already existing section
for para_server.

The patch also kills the unused @server_ldflags@ output variable.

build: Create section for para_audioc.

This moves everything related to para_audioc to the bottom of
configure.ac and kills @audioc_ldflags@.

build: Create section for para_write.

This moves all parts of configure.ac related to para_write to a
separate section at the bottom and kills @write_ldflags@.

The patch also fixes a minor issue for setups where libao is the only
"real" writer. Previously we used FILE_WRITE as the default writer
in this case. This commit changes it to AO_WRITE.

build: Create section for para_play.

This moves all parts related to para_play to a separate section
at the bottom of configure.ac and gets rid of the output variable
@play_ldflags@ in favor of a simple LDFLAGS setting.

build: Create section for para_afh.

This moves those parts of configure.ac related to para_afh to an
own section at the bottom and gets rid of the unused @afh_ldflags@
output variable.

build: Create section for para_recv.

Similar to the previous patch, this moves the creation of
$recv_cmdline_objs and $recv_errlist_objs to a separate section at
the end of configure.ac and gets rid of the now unused @recv_ldflags@
output variable.

build: Create section for para_filter.

Currently, the construction of $filters, $filter_errlist_objs and
$filter_cmdline_objs is spread out over configure.ac as each test
modifies these variables.

The code in configure.ac becomes easier to read if these variables
are set up in a separate section after all tests have been performed.

This patch also gets rid of filter_ldflags. Due to the previous
cleanups, it only contained -lm which is now added in Makefile.in,
along with all other linker flags for para_filter.

build: ldflags conversion: coreaudio.

Straight-forward transformation, relevant to MacOS only.

build: ldflags conversion: libcurses.

Straight-forward as only para_gui depends on the curses lib.

build: ldflags conversion: libnsl.

Similar to the previous patch, this converts the ldflags needed
for nsl and gets rid of the useless check for inet_ntoa().

build: ldflags conversion: libsocket.

This converts the ldflags needed for the socket/connect system calls
in the usual way. It also drops the check for the connect() system
call, as it should always be either in -lc or -lsocket.

build: ldflags conversion: libgcrypt.

Analogous to the previous patch.

build: ldflags conversion: libssl.

Straight-forward transformation.

build: ldflags conversion: libosl.

Straight-forward transformation.

build: ldflags conversion: libsamplerate.

Straight-forward transformation.

build: ldflags conversion: libreadline.

This also renames readline_libs to readline_ldflags, just to make
the naming more consistent to the other tests.

build: ldflags conversion: libao.

build: ldflags conversion: alsa.

Again, a straight-forward transformation.

build: ldflags conversion: liboss.

Straight-forward change, no cppflags necessary.

build: ldflags conversion: libflac.

This converts the handling of the libflac linker flags, just like
the previous patches did for other libraries.

build: ldflags conversion: libmad.

Usual conversion from per-executable variables in configure.ac to
per-library autoconf output variable + per-target LDFLAGS setting.

build: ldflags conversion: libfaad.

This adds the linker flags for libfaad to LDFLAGS in Makefile.in
for those targets that need it. This avoids some duplication in
configure.ac. No cppflags conversion in this patch.

build: ldflags/cppflags conversion: libopus.

Again, the --with-opus-headers option was ignored completely. In fact
opus_cppflags was not even mentioned in Makefile.in.

Fix this and also convert opus_ldflags as usual.

build: cppflags/ldflags conversion: libspeex.

This fixes the --with-speex-headers option, which has never worked,
by adding the speex_cppflags output variable and including the given
flags in CPPFLAGS for the three objects that need it.

The straight-forward conversion for speex linker flags is also
performed in this patch.

build: ldflags conversion: libvorbis.

Same rationale as the previous patch, this time with vorbis_ldflags.

build: ldflags conversion: libogg.

The next step on the way to the removing the per-executable ldflags
variables.  This sets ogg_ldflags in configure.ac, creates an output
variable from it, and adds it to LDFLAGS in Makefile.in for all
executables that need it.

build: cppflags/ldflags conversion: libid3tag

This removes the rule for mp3_afh.o so that this object will instead
be created by means of the generic rule. To make this work, CPPFLAGS is
modified for this target to include suitable cpp options for libid3tag
as determined during configure.

Similarly, the linker flags for libid3tag are removed from
server_ldflags, afh_ldflags, play_ldflags, recv_ldflags and
play_ldflags, and appropriate linker flags are added to LDFLAGS for
all executables that need it.

build: Remove duplicates from $all_objs.

In Makefile.in $all_objs is defined as the concatenation of the
object lists of all executables. This variable is used to determine
the set of dependency files to include. Since the concatenation
contains common objects more than once, we currently include common
dependencies multiple times.

This is not a problem, but it may slow down the build. The make sort
function removes duplicate words, so by sorting $all_objs we make
sure to include each dependency file only once.

build: Remove inactive DEBUG_CPPFLAGS.

These were commented out for some years, and don't really add any
value.

build: Replace error2.pl by error2.c.

The C code is much faster than the old perl script. Since the resulting
executable runs on the build system, the compiler for this system
must be called to compile error2.c. The new HOSTCC variable takes
care of this.

With this patch applied, the only remaining dependencies on perl are
help2man and autoconf.

build: Fix filter output of configure summary.

The $filters variable contained newlines, which made the output
rather ugly.

build: Convert remaining object lists to one object per line format.

This makes patches easier to read and eases conflict handling.

build: Separate curses detection from gui configuration.

It's cleaner to separate feature/library detection from the logic
that decides whether an executable can be built.

This patch creates a separate section for para_gui which contains
the list of all objects for para_gui and sets $build_gui to either
"yes" or "no", depending on whether a usable curses library was found.

build: Move down fade section.

The logic which checks whether an executable can be build on the
given system should come after the tests for features and libraries.
The section for the para_fade executable violates this rule, so move
it down to where it belongs.