mpdec.c: %d -> %zd conversion

This fixes some warnings of the form

mp3dec.c:66: warning: format '%d' expects type 'int', but argument 5 has type 'size_t'

on MacOs.

kill para_client's interactive mode

Again, it is not worth the compatibility trouble it causes.  It's much
better to implement such functionality with a bash script or similar.

Also, add Gerd Becker to CREDITS and mention that paraslash compiles
under MacOS.

configure.ac: Add check for unix socket credentials

remove null termination.

The previous patch missed to remove a line. Quite a serious mistake as
'ret' no longer contains the length of the message but the user id..

avoid struct ucred in audiod.c

This is a preparation for making ucred optional. It moves everything
which uses unix socket credentials into the single source file net.c.

recv_cred_buffer typo: Return proper error number

kill mallinfo

It is not important enough to care about the compatibility
issues.

Rename MIN, MAX, ABS macros

MIN and MAX are already defined on MacOS.

gcc-compat.h: rename __unused to __a_unused

__unused is already defined on MacOS.

gcc-compat: Kill __pure

It is not used within paraslash and is already defined on MacOS.

com_sa: pedantic mem leak fix.

In fact, this memory leak doesn't matter much because it happens in the child
which calls exit shortly afterwards. But it's considered good programming
practice to free all memory, so..

com_cs: Return a syntax error if argc > 2.

Previously, argv[i] was silently ignored for i > 1.

mysql selector commands: escape argv[] when used in a query.

typo

This was introduced in the previous patch.

improved version of split_args()

Thanks to Lorenzo Bettini for pointing this out.

This patch also fixes a bug with the default filter configuration which
allocated too little memory for the array of filter configurations. Now
we always allocate space for at least three entries.

para_server/para_audiod: new option --group

Used to set the GID at startup. This patch also reformats
server.ggo and audiod.ggo to make them more readable.

further header cleanup

new function: set_stream_fds()

There are many places like the following:

FD_SET(fd, fdset);
max = MAX(max, fd);

set_stream_fds() simply combines these two commands.

small header cleanup

Add documentation for para_select() and mark_fd_nonblock()

This patch also fixes a thinko in para_select().

introduce mark_fd_nonblock()

Calls fcntl(2) to prepare an fd for nonblocking file operations. Also, make
fd.c and para_gui use the error subssytem.

introduce para_select()

A simple wrapper for select(2) that checks for EINTR and restarts the
select call in this case.

New source files: fd.c/fd.h

ATM, they only contain the file_exists() function which was pulled
out from exec.c. More to follow.

signal.c: mark the read and write ends of the signal pipe non-blocking.

rename the array of supported audio file senders from 'dblist' to 'selectors'

FEATURES: (no log message)

fix potential segfault in para_client

The first data buffer which is sent by the server is checked for
the AWAITING_DATA_MSG, but that buffer might not be 0-terminated..

Some other minor cleanups also.

mp3: Increase eof timeout

This fixes some problems with ortp.

Fix use of uninitialized variable 'now' in afs_preselect()

db.h: fix documentation

The pre_select function of struct audio_file_selector must _return_
the highest-numbered file descriptor.

Minor audiod mainloop cleanup

stat_client_write(): add help text for new 'itemnum' parameter

update audiod help text

Replace the old --anticlip and --volume options in the example.

specify the three grab modes as values in grab_client.ggo

That makes gengetopt check that the argument to the --mode
option is one of sloppy, pedantic, or aggressive, so get rid of
E_INVALID_GRAB_MODE, the gc_modes array and of the error check in
grab_client.c.

Another nice side-effect is that also "s", "p", "a" work as
abreviations for the grab mode argument.

Reformat grab_client.ggo as we are at it.

gengetopt: use the new (and more descriptive) 'optional' instead of 'no'.

fix grab -V

Currently, -V is silently ignored, i.e. "grab -V" dumps binary data
to stdout which probably doesn't meet the expectation of the user ;)

gengetopt improvements

Use the --set-package option of gengetopts for receivers, filters
and the grab command, and set the 'package' name appropriately.
Moreover, --no-handle-version is needed for the grab command. Without
it, grab -V terminates para_audiod.

audiod: use the new help text array of gengetopt-2.17

This allows to get rid of an ugly hack for the grab command.

update to gengetopt-2.17

gengetopt-2.17 has an improved help management, thanks again to Lorenzo
Bettini for implementing my suggestions. The help texts of the options
are contained in an array which is defined in xyz.cmdline.c and declared
as extern in xyz.cmdline.h.

In order to link more than one xyz.cmdline.o into a single binary,
these arrays must have different names. So use the --arg-struct-name
option of gengetopt for each receiver/filter.

update filter help text

The example used the --anticlip and --volume options of the compress
filter which are no longer present. Replace these by valid command
line options.

audiod: boost the stat command

It now takes any number of arguments, including zero which selects
the old behaviour (dump _all_ status items to stdout).  If called
with arguments, each arg must be a valid status item and only these
are written to sdout.

stat_line_valid(): use the FOR_EACH_STAT_ITEM macro

audiod: switch from ringbuffer to statically allocated array

This patch also fixes a minor bug where audiod would send empty status
lines or lines containing only a colon (only happens if para_server
status is paused).

doxify dccp

Add documentation for all non-static functions, and move the defines
of the magic dccp constants from dccp.h to dccp.c as they are only
used there.

Add -Wwrite-strings to CPPFLAGS

Commented out for now.

use const qualifiers where possible

mysql_selector.c: whitespace cleanup

audiod: Kill another unused variable

ortp_send: Kill an unused variable

mysql selector: Fix a memory leak in get_query()

If the stream definition contained N > 1 "score:" lines (an invalid
configuration anyway, there should be at most one), we leaked the first
N - 1 score lines. The fix is to take into account the first one only.

com_next() fix.

Do not clear the AFS_NEXT flag in afs_send_chunk() and increase mmd->events.
Fixes a (rare) problem where the next command did not work.

s_a_r_list(): return NULL on errors

And fix the documentation.

audiod: use set_restart_barrier() instead of duplicating it.

dccp_recv: increase output buffer size

dccp_send: fix sending of header packets

The current code sends the header before _each_ packet, rather than only
once. Fix is trivial.

dccp: avoid "message too long" errors.

Break down the write into 1KB chunks as a workaround.

log the IP address of the connecting client.

kill duplicate write_ok()

use the generic inline function from send.h instead and shutdown the
connection if an error occured in write_ok().

Fix return value of dccp_recv pre_select()

Also, add a log message on errors/eof.

gui: Only check STDIN for readability if curses is active.

More %d -> %zd fixes

Noted by Steffen Klassert.

ortp: make jitter compensation configurable.

Non-zero values turn on ortp's adaptive jitter compensation.

dccp_send: comment out noisy debug message.

fixes for ortp-0.9.1

turn on adaptive jitter compensation. ortp-0.9.1 seems to be unhappy
without it. This patch also contains some simplifications and timing
improvements.

db.c: use PARA_STRERROR()...

... instead of logging the error number only.

update README

Mention the new dccp sender/receiver.

add dccp.h to tarball

The tarball did not compile because dccp.h was missing.

update FEATURES

Mention the new dccp sender.

update to libortp-0.9.1

glib was dropped from libortp, and paraslash does not need it
either. So kill the check for glib in configure.ac and no longer
link para_server, para_recv, and para_audiod gainst glib. Moreover,
change guint32 to uint32_t in ortp_recv.c and ortp_send.c.

make install: do not create an unused directory

$(VARDIR)/fifo is unused for ages.

update NEWS

Fix help text of --list_receivers.

It still used the old syntax with colons as separators.

speed up the compress filter

Complete rewrite of the core loop of the compress filter.  The gprof
output of para_audiod (see below) indicates that CPU usage goes down
to about 40%.

new:
~~~~

Each sample counts as 0.01 seconds.
%   cumulative   self              self     total
time   seconds   seconds    calls   s/call   s/call  name

35.94      1.47     1.47     7414     0.00     0.00  compress
22.74      2.40     0.93    57319     0.00     0.00  mp3dec
16.87      3.09     0.69        1     0.69     4.04  audiod_mainloop

34.59      1.47     1.47     7414     0.00     0.00  compress
22.59      2.43     0.96        1     0.96     4.20  audiod_mainloop
17.53      3.17     0.74    61285     0.00     0.00  mp3dec

35.41      1.48     1.48     7414     0.00     0.00  compress
20.10      2.32     0.84        1     0.84     4.15  audiod_mainloop
17.82      3.06     0.74    58675     0.00     0.00  mp3dec

Average of self-seconds (3rd column):
compress 1.47
audiod_mainloop 0.83
mp3dec 0.80

0.2.11:
~~~~~~~

Each sample counts as 0.01 seconds.
%   cumulative   self              self     total
time   seconds   seconds    calls   s/call   s/call  name

56.38      3.31     3.31     7414     0.00     0.00  compress
12.93      4.08     0.76    57566     0.00     0.00  mp3dec
11.39      4.75     0.67        1     0.67     5.86  audiod_mainloop

57.59      3.38     3.38     7414     0.00     0.00  compress
14.08      4.20     0.82    63095     0.00     0.00  mp3dec
11.26      4.86     0.66        1     0.66     5.82  audiod_mainloop

57.79      4.12     4.12     7414     0.00     0.00  compress
13.62      5.08     0.97        1     0.97     7.07  audiod_mainloop
 8.08      5.66     0.57    57196     0.00     0.00  mp3dec

Average of self-seconds (3rd column):
compress  3.60
audiod_mainloop 0.76
mp3dec 0.71

Merge branch 'dccp'

Fix up merge conflicts by hand.
Conflicts:

configure.ac
error.h
server.ggo

new codename, reset version to git

the paraslash-0.2.11 release tarball

paraslash 0.2.11

whitespace cleanup

update NEWS

deleted:  index.html

It crept in during the initial commit, and it is long outdated.

mysql_selector.c: rename get_dbinfo() to get_selector_info()

mysql_selector.c: rename refresh_mmd_dbinfo() to refresh_selector_info()

kill a macro which is only used once.

Insert the text directly instead.

mmd: rename dbt_change to selector_change

mmd: rename dbt_num to selector_num

mmd: rename dbinfo to selector_info

Some more (trivial) "database tool" -> "audio file selector" conversions

dbtool -> selector update for web documentation web page

More dbtool -> selector updates

rename mysql.c to mysql_selector.c

Just to be consistent.

README.mysql: cdt -> chs update

Rename E_BAD_DBTOOL to E_BAD_SELECTOR

Rename random_dbtool.c to random_selector.c

rename plm_dbtool.c to playlist_selector.c

Rename "dbtool" status item to "selector"

Change doxygen file description from dbtool to selector

Another trivial rename

gui,sdl_gui: change the prefix from "dbtool" to "selector"

trivial dbtool->selector renames

Rename some local variables in command.c