Merge branch 't/misc_cleanups'

Was cooking since 2013-08-04.

3f97ae buffer_tree: Improve documentation of btr_no_parent().
fb9209 Avoid gcc warning on FreeBSD.
778af7 command.c: Avoid declaration after statement.
61c957 extract_v4_addr(): Don't return a structure.
1d7937 command.c: Remove pointless initialization.
c17ede crypt.c: Remove condition which is always true.
a12ecb ipc: Close proc fd.
01d37b Make log functions static.
0ba631 score: Improve documentation of score_update().
d6cf71 GIT-VERSION-GEN: Don't include output of git update-index.

Merge branch 'maint'

This incoporates a couple of important bug fixes found at athcx.

bash_completion: Run client/audioc from PATH.

This way it works for everybody.

configure: Print opus audio file handler if opus lib was found.

The variable is called $audio_format_handlers rather than
$server_audio_formats. The value is only used to display the supported
audio formats, so this bug is rather benign.

btr: splice-out fix.

When splicing out a node we set the ->parent pointer of all child
nodes to the parent of the given node, and move each child to the
->children list of the parent.

Except when there is no parent. If the given node was a root node
(or an internal node whose parent vanished), we leave the ->children
list untouched. In this case the assertion a few lines later triggers
and aborts the program. Fix this by removing the nodes from the list.
Such nodes have become root nodes themselves, so they should not be
on any list of children.

fecdec: Avoid fecdec output buffer overruns.

The size of the buffer tree pool of the amp filter is 64K, which
is smaller than BTRN_MAX_PENDING of buffer_tree.c (96K). The latter
value is used in btr_node_status() as follows. If input for a buffer
tree node is available and there is less than BTRN_MAX_PENDING bytes
in the output queue of the node, the function returns 1 to indicate
that the node should continue to process its input.

This can result in a fatal error condition when the buffer tree pool
fills up completely. Avoid this by increasing the pool size to 128K.

vss: Avoid assertion "i == g->num_header_slices - 1".

This fixes a very old bug in the timing code for fec slices. Depending
on the number of bytes to submit and on the number of slices of the
current fec group it may happen that we reserve more slices than
needed for the audio file header. This is not a problem per se, but the
above assertion hits in this case and aborts para_server.

Fix this by removing the assertion, and make sure that all reserved
header slices are properly initialized.

configure: Fix restoring LDFLAGS.

The configure script saves the values of CPPFLAGS, CFLAGS and LDLFAGS
before each check and restores the old values afterwards. However,
for LDLFAGS this was broken at several places, since we used LD_FLAGS
(with an underscore) for saving the value and LDFLAGS for restoring it
(or vice versa).

This bug had the effect that a user-specified value of LDFLAGS was
ignored in some tests because LDFLAGS was restored to the empty
value LD_FLAGS during an earlier test.

This patch changes all occurrences of LD_FLAGS in configure.ac to
LDFLAGS.

error.h: Fix typo in comment.

amp: Downgrade level of log message.

The scaling factor is often one, so this is not very important
information.  Set the loglevel to INFO.

command: Make server_cmds[] and afs_cmds[] static.

These arrays are only used in command.c.

Don't include $codename in ggo version string.

This variable no longer used. Since we do not rely on gengetopt's
implementation of --version any more, this does not matter much.
Let's fix it anyway.

doc: Add man page of para_play to documentation web page.

In December 2012 para_play was merged to master but we missed to
update the web page accordingly.

buffer_tree: Improve documentation of btr_no_parent().

Avoid gcc warning on FreeBSD.

FreeBSD 9.1 ships a patched gcc-4.2.1 which complains about use of
an initialized variable:

fd.c: In function 'for_each_file_in_dir':
fd.c:728: warning: 'dir' may be used uninitialized in this function

This warning is bogus since para_opendir() only leaves the dir
pointer unset on failures. But in this case for_each_file_in_dir()
returns early without using the pointer.

This commit changes para_opendir() to set dir to NULL before doing
anything else. This avoids the warning at almost no cost.

command.c: Avoid declaration after statement.

Fixes

command.c:224:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]

extract_v4_addr(): Don't return a structure.

When compiling with -Waggregate-return, gcc issues the following warning:

net.c: In function 'extract_v4_addr':
net.c:682: warning: function returns an aggregate
acl.c: In function 'acl_check_access':
acl.c:60: warning: function call has aggregate value

It's not entirely clear how bad it is these days to return a structure, but
extract_v4_addr() is the only function in the tree which does it. Since it
can be easily changed to take a pointer instead and there is only a single
caller, let's implement it this way.

command.c: Remove pointless initialization.

Found by the clang analyzer:

command.c:934:16: warning: Assigned value is always the same as the existing value
for (i = 0, p = iov->iov_base; p < end; i++)
    ~ ^ ~~~~~~~~~~~~~

clang is right: A couple of lines before the for loop we set p to
iov->iov_base and do not modify it.

crypt.c: Remove condition which is always true.

rsa can never be NULL here.

ipc: Close proc fd.

We read shmmax only once, so there is no point in keeping this
file descriptor open forever.

Make log functions static.

All PARA_XXX_LOG() macros use the public para_log function pointer
for quite some time now, so the actual implementations of the various
log functions can be made static.

score: Improve documentation of score_update().

Remove a stray "and" and the list of possible errors, which is not
very useful and possibly incomplete.

GIT-VERSION-GEN: Don't include output of git update-index.

GIT-VERSION-GEN calls git update-index which might write something
like "configure.ac: Needs update" to stdout. We don't want this text
to be part of the version string, so make sure we discard the output
of git update-index.

NEWS: Update download links to 0.5.0.

reset version to 'git'

paraslash 0.5.0

Merge branch 't/web_improvements'

The new pages have been tested on slnx with no problems, so they're
ready for prime time.

329719 web: Move menu down a bit.
c6ac6c web: Make image rule global.
c53634 Replace main page by NEWS.
3c2dfc NEWS: Replace "to be announced" text.
c2ea0c web: Add tarball and signature link of recent release to NEWS.
b35afc web: Adjust sizes of monospace fonts.
7fb28a web: Avoid gray area on gitweb page.
f3f978 web: Move important parts of the style sheet to the top of the file.
57058e web: Remove some duplicate rules in para.css.
5005fb web: Combine the two .css files.
d74c89 web: Add header to gitweb and API reference sites.
076b70 web: Increase size of log messages and diffs.
0384fd web: Unify link colors.
893669 web: Fix gitweb link on doxygen pages.
607d79 web: Unify <br> elements.
40ceca web: Remove valign tag in header files.
37bfc1 web: Further unify header.html, header2.html and gitweb_header.html.
197add web: Unify css rules for tables.
21422a web: Remove horizontal line at the top of the page.
53963d web: Remove style element in html.
4c061d web: Make the body selector of the two css files consistent.
f9fadc web: Use lower-case css selectors.
22fb31 web: Increase font size.
2f201d web: css whitespace fixes.
51445c web: Remove commented-out sections of css files.
71d445 web: Remove border="0" tag.
02c1a5 web: Don't set border, cellpadding and cellspacing.
6e0b01 web: Remove basefont tag.
42a4cd Update README.
2210c6 web: Remove explanation of the paraslash acronym.

Merge branch 't/image-0.5'

After almost three months we finally got there.

8b7b77 The overwhelming new logo for paraslash.0.5.

overview.pdf: Fix overfull lines.

The Ubuntu version of dia (which is used to create the public file
on the web page) seems to behave differently than the self-compiled
version used on my development box.

This resulted in a rather strange looking pdf. Fix this by manually
adjusting lines using the Ubuntu version.

test suite: Change underscores to dashes also in tests script options.

The previous commit broke t0004. Fix is trivial.

Change all multi-word options to dashes instead of underscores.

Currently some of these options use dashes to separate the words
while others use the underscore character instead. The auto-generated
--detailed-help option contains a dash, so the only consistent way
is to use dashes everywhere.

Hence this commit replaces all underscores in option names by dashes
and updates the documentation accordingly.

Fortunately, this does not require to change the code, because
gengetopt translates dashes into underscores in the generated C code.
It is, however, an API change, so this is 0.5.0 material.

Merge branch 't/dia'

Was cooking since 2013-06-07.

f5fb7a Overview: Add new slides, powered by dia.

Merge remote-tracking branch 'refs/remotes/bthcx/master'

Merge branch 't/setatt_fnmatch'

Was cooking for about two weeks and seems to work fine.

73299e com_setatt(): Allow to specify a file name pattern.
fd5f94 Move com_setatt() from attribute.c to aft.c.
a8573c com_setatt(): Use get_attribute_bitnum_by_name().
47dbd9 Mark buffer pointer of pass_buffer_as_shm() as const.

FEATURES: Mention ogg/opus.

Support for ogg/opus was added in 0.4.13, but we missed to update the
FEATURES text.

Merge branch 't/m4_deps'

This branch was cooking since 2013-08-04 and no problems showed up.

e249d1 Generate ggo dependencies automatically.
8bf727 Add para_ prefix for executables in Makefile.in.

web: Move menu down a bit.

This looks better. It can easily be achieved by adding a <br> tag
before the menu.

web: Make image rule global.

This way it also applies to the paraslash logo. Without this, old
mozilla browsers show a margin around the image, which looks ugly.

Replace main page by NEWS.

The main page listed almost only the releases. This information is
also available through the NEWS menu, so remove the main page and
make NEWS.html the new index.html.

This commit also reorders the menu items and adds the new "Development"
item. This page contains the information on the topic branches which
was previously known as cooking.html.

NEWS: Replace "to be announced" text.

The new text mentions that the changes listed here have already
been merged to the current master branch, which is more valuable
information.

web: Add tarball and signature link of recent release to NEWS.

This is a first step to replace the main page.

web: Adjust sizes of monospace fonts.

120% is big enough, let's use this size everywhere.

web: Avoid gray area on gitweb page.

Removing this section makes it go away.

web: Move important parts of the style sheet to the top of the file.

web: Remove some duplicate rules in para.css.

web: Combine the two .css files.

Having only one style file avoids to duplicate the rules for common
elements.  This is the result of cat gitweb.css >> para.css plus a
minor fix.

gitweb_config.perl on the destination site, and para_adm have to be
updated at the same time to make this work.

web: Add header to gitweb and API reference sites.

This makes the layout of these two pages identical to the main pages.

web: Increase size of log messages and diffs.

The size of these was hardcoded to 12px.

web: Unify link colors.

The color on the gitweb page was slightly different.

web: Fix gitweb link on doxygen pages.

This did not work on the test instances (it did work on the official
site though).

web: Unify <br> elements.

This caused the menu of the gitweb page to occur at a slightly different
location, which was somewhat annoying.

web: Remove valign tag in header files.

It does not belong there and seems to have no effect.

web: Further unify header.html, header2.html and gitweb_header.html.

Indentation was messed up, and a title tag was missing.

web: Unify css rules for tables.

web: Remove horizontal line at the top of the page.

The gitweb page does not have this line, and the page looks better
without it.

web: Remove style element in html.

web: Make the body selector of the two css files consistent.

web: Use lower-case css selectors.

In para.css most css element selectors were in upper case, while
gitweb.css used lower case selectors. This patch makes the two
consistent.

web: Increase font size.

The default value of font-size is medium, which seems to be more appropriate.

web: css whitespace fixes.

Indentation was rather inconsistent.

web: Remove commented-out sections of css files.

web: Remove border="0" tag.

Again, this has no effect.

web: Don't set border, cellpadding and cellspacing.

It has (almost) no effect.

web: Remove basefont tag.

This is not needed at all.

Update README.

The README file mentions mp3, ogg/vorbis, aac and wma, but speex,
flac and ogg/opus were all missing.  Do not even try to list the
supported audio formats, it always gets out of sync.

Secondly, since this file will not be processed by grutatxt, it's
pointless to format it according to the grutatxt syntax.

Finally, this change adds the URLs of the web page and the git
repository, as well as my email address.

web: Remove explanation of the paraslash acronym.

It's kind of lame, so get rid of it.

Conflicts:
README

Merge branch 't/remove_sb_compat'

This topic branch was cooking for over 4 months and should now be
ready for prime time. The patches break compatibility with earlier
versions, so we need to bump the version number real soon now.

72ffa7 client: Only start stdin task for addblob commands.
85a6ae Remove old stream cipher API.
d1407f Allow addblob commands to create output.
243e31 client: Remove client_recv_buffer().
277ed4 client: Remove sb-compatibility code.
36ba18 Reject non-sideband connections.
6ca50b blob: Avoid zero sized memcpy().
cf308c Fix typo in documentation of stdin_command().
fdb9d2 blob: Simplify fd2buf().

com_setatt(): Allow to specify a file name pattern.

The help text of the setatt command claims that file name patterns
may be specified as arguments. The text also contains an example which
involves a wildcard pattern. However, this never worked as advertised
since the command actually accepts only full paths.

This commit teaches the command to process the given arguments as
patterns by employing the same pattern matching loop that is also
used by other commands.

In addition we now also send a "no matches" message to the client
in case there was no match. Previously the command stayed silent and
terminated successfully in this case.

Move com_setatt() from attribute.c to aft.c.

This command operates on rows of the audio file table. Pure code
movement, no changes.

com_setatt(): Use get_attribute_bitnum_by_name().

We have this nice helper, so don't open-code it.

Mark buffer pointer of pass_buffer_as_shm() as const.

Generate ggo dependencies automatically.

Currently we maintain the dependency list of each ggo file manually in
m4/gengetopt/makefile. This is tedious and error-prone, and missing
dependencies result in build failures which are not repeatable and
hard to locate.

This changes the build system to keep track of these dependencies
automatically. We now create one .m4d file for each executable. This
file lists the dependencies of the corresponding ggo file, i.e. those
m4 files which are included in order to make the ggo.

All .m4d files are pulled in from the main Makefile via the -include
directive, in the same way we include the object dependency files
(.d) generated by gcc.

Add para_ prefix for executables in Makefile.in.

Using make's $(addprefix) function in Makefile.in is simpler than
implementing our own version in configure.ac.  Moreover, in Makefile.in
we now have access to both the prefixed and the un-prefixed list
of executables.

server/gcrypt: Fix sending the empty status items.

When para_server enters the "stopped" state, it needs to inform all
clients which are executing the stat command that no audio file is
currently available.

To this aim the stat command handler calls empty_status_items(), which
creates a buffer containing empty values for most status items. This
function tries to be smart by computing the buffer only once. It
saves a reference in a static variable so that on subsequent calls
it can simply return the same buffer.

This works fine when para_server is compiled against the openssl
crypto library. However, it fails when libgcrypt is used because with
libgcrypt the send buffer is encrypted in-place. Hence on subsequent
calls the already encrypted buffer will be encrypted again, resulting
in garbage being sent to the client.

This patch avoids this bug by using a fresh buffer each time the
empty status items are sent.

new codename, reset version to git

paraslash 0.4.13

time.c: Make declarations match definitions.

Also fix some whitespace issues.

Add two missing GPL and doxygen file headers.

It's a pity that doxygen doesn't warn about this.

sched.h: Remove outdated comment.

The term "these functions" is undefined here, and setting t->error
is deprecated.  The post_select() functions are supposed to return
an error code instead, but this is already mentioned further down in
the file, so let's just remove the comment.

Fix documentation of filter->post-select().

The ->post_select() methods return an integer these days and should
*not* set the ->error field of the task struct directly.

ipc.c: Fix typo in comment.

aft.c: Remove a stray "*" in a comment.

Move com_select_callback() closer to com_select().

It's considered good style to have the command handler and it's
callback next to each other. No actual changes.

buffer_tree.c: Add documentation of btr_parent().

This function was introduced in commit 37e0dfe7 (check_wav: Ask parent
nodes before falling back to defaults) without documentation.

ggo.c: Document return value of printf_or_die().

Commit b59a3c41 (filter: Wrap lines in the available filter list)
changed the return value of this function from void to int but missed
to update the documentation.

Mood: Document return value of moods_event_handler().

Just to make doxygen happy.

net.c: Add documentation of flowopt_new().

It's not obvious that the returned structure does not need to
be freed by the caller, so let's document this fact.

net: Remove unused flowopt_add_bool().

recv.h: Fix comment indentation.

Remove some duplicate semicolons.

Remove some unused error codes.

Quite a few error codes have become unused, but the stale entries
of error.h remained. Unfortunately, we have no way to spot these
automatically.

Doxyfile: Predefine HAVE_CLOCK_GETTIME.

Fixes the following doxygen warning:

time.c:227: Warning: include file sys/time.h not found, perhaps you forgot to add its directory to INCLUDE_PATH?

audiod: Close filters in reverse order.

Otherwise, a subsequent filter might refer to a buffer reference of
the previous filter, accessing freed memory. This is not a bug right
now, as audiod only closes filters after the full buffer tree has
become inactive, i.e. after all buffer tree nodes have unregistered
their task. It's cleaner to reverse the loop though, and it has no
additional cost.

opusdec: Get rid of opusdec_pre_select().

The generic one should do just fine and the additional 100ms timeout
should not be necessary.

opusdec: avoid __STDC_VERSION__ warning.

opusdec_filter.c indirectly includes opus_types.h which checks whether
__STDC_VERSION__ >= 199901L. However, at least some gcc versions
don't define __STDC_VERSION__ which results in

warning: "__STDC_VERSION__" is not defined

For example, this happens on Ubuntu lucid, which ships gcc-4.4.3.

This patch gets rid of the warning by defining __STDC_VERSION__
if necessary prior to including the opus_types header.

afh: Initialize audio format handlers only once.

afc.c calls afh_init() twice. This does not really hurt as this
function is idempotent, but it causes the initialization log messages
to be printed twice.

This gets rid of the additional call to afh_init() which crept in
in commit 042767ce (Use self-made help to avoid recompilations on
version changes).

mp3_afh: Use symbolic constants for id3 tag identifiers.

Merge branch 't/opus'

Was cooking for two weeks with no problems.

273756 The opus decoder.
7007ae The opus audio format handler.
8bcf75 ogg/opus: Infrastructure.
53133e speex: Don't export spx_ctl().

Merge branch 't/versioning_improvements'

Cooking for ~2 weeks.

15e99a version.c: Mark version_git() as const.
9bdebf Remove CODENAME macro.
be2f6b gui: Use version_single_line().
d60dae Improve man page layout.
5dbc9a afh/play: Include supported audio formats in help output.
042767 Use self-made help to avoid recompilations on version changes.
48f1fc Provide "purpose" texts.
aa74a9 Revamp ggo help.
9f7a49 afh_recv: Replace ggo text section by description.
b59e0e Make gengetopt descriptions work.
06b3e7 Introduce version.c to limit recompilation on version changes.
75feac Make all commands print git version and improve version string.
b01605 Avoid unwanted log messages during startup.
625fdb Don't check return value of command line parsers unnecessarily.
533b03 Build receivers, filters and writers without -h and -V support.
b59a3c filter: Wrap lines in the available filter list.
9e56d3 audioc: Print config file errors.
e5264d doc: Rewrite udp sender description
5ec373 client: Fix typo in comment.
6d5159 client: Remove duplicate include.

version.c: Mark version_git() as const.

Fixes the following warning on gcc-4.8.1:

version.c:13:13: warning: function might be candidate for attribute 'const' [-Wsuggest-attribute=const]