cosmetics

check barriers also in afs_send_chunk()

As this function is called even if the select timeout didn't expire,
we'll have to check the eof_barrier and the data_send barrier also
early in afs_send_chunk() and return if there's time left.

ortp_send: Fix EOF packet sending

server: call afs_send_chunk() even if no timeout occured.

afs_send_chunk() checks the current time anyway.

ortp_send: kill useless gettimeofday()

fix error log message

Also, make recv_buffer() write a zero even if the underlying recv_bin_buffer()
failed.

Send correct http message.

Screwed up since the 1.0 vs. 1.1 fix.

grab: init gc->audio_format_num to -1

Previously it was 0, which happens to be the mp3 audio format number.
Therefore, grab without args only grabbed mp3 files. This patch makes
it grab any audio format.

audiod: fix minor memory leak

The default filter setup code (which is only called at startup)
leaks four bytes.

compress: Adjust forumla for clipping

Use (3 * old + min) / 4 instead of (old + min) / 2 as the new gain
if clipping occures.

Doxify the write subsystem

audiod: reset max_fileno at each run.

Kill some noisy debug messages also.

com_upd: always escape the filename

Fixes problems with filenames starting with "'"

grab_client: Error out if given an invalid grab mode

com_upd: ignore duplicates

Previously the update command aborted if files with duplicate basenames
were found. Suggested by Gerd Becker.

fix trivial compiler warning in net.c

net.c:347:7: warning: extra tokens at end of #else directive

configure: add two mysql-related command line options

Add "-I ." to aclocal options

This way, one can put missing m4 files (like pkg.m4) to the paraslash
directory.

mp3dec: skip invalid frame headers

First try to decode the header, and if that fails with a recoverable
error, return FRAME_HEADER_SIZE to skip the invalid frame.

net.c: fix ifdef UCRED madness

simply use ordinary recv_buffer()/send_buffer() in case ucred is
not available.

add a notice message if the expected pattern was not received

1000 * 1000 microseconds is too much for Darwin

linux never complained about that, but Darwin returns -EINVAL.

HTTP_GET_MSG: Do not resist on http 1.0

recent itunes for example sends 1.1. The obvious solution is to just
accept every client that sends at least 'GET / HTTP/'. Spotted by
Gerd Becker.

Fix another gcc signedness warning in write.c

minor mysql_selector cleanup.

make_message() never returns NULL, so remove the check.

alsa_writer.c: Fix gcc-4 signedness warning

NEWS: Mention the reason for size decrease of the tarball

http_send.c: Add a comment

dccp_send.c: Set status to SENDER_OFF on errros.

fade.c: whitespace cleanup

INSTALL: Add more explanations for unexperienced users.

Makefile.in: Fix generation of web pages

use git-tar-tree to make the tarball

This simplifies Makefile.in quite a bit, speeds up the tarball
creation, and reduces the size of the compressed tarball from 560KB
to 408KB, mostly because it no longer contains the screenshot images.

Simplify wng_write()

by killing the variable "p" which is only used once. Fixes

warning: pointer targets in passing argument 1 of 'wn->writer->write' differ in signedness

write.c: fix more compiler warnings on MacOs

format '%d' expects type 'int', but argument 4 has type 'size_t'
pointer targets in passing argument 1 of 'read_stdin' differ in signedness

command.c: cast members of struct timeval to long unsigned

This should fix

command.c:441: warning: format '%lu' expects type 'long unsigned int', but argument 31 has type 'suseconds_t'

on MacOs.

remove 1.0

there's not so much left..

Makefile.in: remove unused variable CONFDIR

More doxygen updates

add doxygen file comments

modified: alsa_writer.c write.h write_common.c write_common.h

write.c: Make check_args() static

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.