Doxyfile: replace play.c by write.c

add --disable-ortp option to configure

This is handy if an old ortp library is installed.

Makefile.in: kill sample_conf variable

There are no more sample configs left. No sample.conf is better than
a notoriously outdated one..

remove gui.conf.sample

didn't even know it still existed.

README: mention that gcc-2.95 no longer works.

Makefile.in: activate even more gcc warnings

Add -Wmissing-format-attribute -Wunused-macros (these are unknown
to the now unsupported gcc-2.95).

write.h: mark two variables extern

ortp_recv compute_next_chunk(): kill unused parameter now

ortp_recv.c post_select: mark unused paramters with __a_unused

kill unused BUF_TO_VAL macro

Makefile.in: Remove special treatment of ortp_send/ortp_recv

they no longer depend on glib, so use the generic rule for building
object files. This made gcc spit out some trivial warnings about
unused variables which are also fixed in this patch.

Fix typo in gcc-compat.h

It only affects gcc-2 and non-gcc (which is untested anyway).

net.c: make it compile without ucred

This patch is ugly as hell and only compile-tested. It obviously
needs more work (or a totally different approch like using the usual
send/recv functions in case ucred is not available).

MacOs testers are welcome.

README updates

Update info on required gcc version in README, small README.mysql update.

audioc.c, audiod.c, server.c: use para_fd_set()

cosmetics

NEWS: gcc-2.95 is no longer supported

The admin of the single remaining system I have access to which was
still running gcc-2.95 recently upgraded to gcc-3 (a version which
is merely 3 years old rather than the 7 years old gcc-2.95).

This means that upcoming paraslash releases will no longer be
compile-tested with gcc-2.95.

gcc-2.95
finally didn't manage to survive
it served all well for all the time
but no one really cares to whine
as we all meet in some other life

use para_fd_set also in gui.c and in http_send.c

Makefile.in: kill unused target para_compress

para_compress went away in paraslash-0.2.8..

kill unused E_LIST_WRITERS_GIVEN

Quote the the strings in WRITER_NAME correctly.

That fixes problems on debian.

Makefile.in: add tow missing headers for the tarball

With this fix, the tarball compiles again.

reformat and improve write.ggo

add more documentation for the --writer option, reorder options
and move alsa-specific options to an own section.

para_play -> para_write documentation update

split play.c and rename para_play to para_write

This separates the code to write an audio stream into several
independent pieces:

write.h: definition of writer-related structures
write.c: the para_write main program
write_common.c: functions to be shared between para_write
and para_audiod
write_common.h: exported functions of write_common.c
alsa_writer.c: writes the stream to an alsa sound device
file_writer.c: writes the stream to a file

Other writers (aka output plugins) can be added easily.

add new writer to para_play: file

It simply writes the input to a random filename under ~/.paraslash.
Also, add some writer-related macros to config.h and two new options
for para_play: --writer and --list_writers.

Further para_play abstraction: struct writer_node_group

This structure contains the list of writer nodes and information common to all
of these nodes. The idea is that the wng group code will be shared between
para_audiod and para_play.

para_play: simplify play_pcm(), add E_PLAY_OVERRUN, E_PREMATURE_END

switch to the new struct writer/writer_node abstraction

There's only one writer possible ATM, but it is designed to allow
more than one.

This patch also, adds a loglevel config option to para_play.

introduce struct writer and struct writer_node

They are unused for now.

alsa_shutdown(): free struct private_alsa_data

para_play: introduce struct private_alsa_data

this structure combines all alsa specific state variables of
para_play. Its usage separates alsa specific stuff from the
generic code of para_play.

make chunk_size a local var and rename it to period_size

make para_play use the error subsystem

play.c: Let alsa_init() return the number of bytes per chunk

play.c: make alsa_write() take the number of _bytes_ to write

rather than the number of frames.

play.c: reorder and rename some functions.

Move everything which depends on ALSA to the three functions
alsa_init(), former set_alsa_params(), alsa_write(), former
pcm_write(), and alsa_shutdown().

replace memcpy by memmove

as the areas in question may overlap

Fix some typos in CREDITS

Kill noisy debug message

audiod: do not use pointers to members of struct timeval for sscanf()

Use temporary long unsigned variables instead. Fixes warnings of the
form

audiod.c:844: warning: format '%lu' expects type 'long unsigned int *', but argument 4 has type 'suseconds_t *'

on MacOs

Typo

fix more struct timeval related warnings

Use tv2ms() which returns long unsigned where appropriate.
For the remaining cases, cast the members of struct timeval
to long unsigned if used via a format string.

audiod: use tv2ms to show the time difference

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.