ogg_afh.c: Kill global variable num_chunks

The code is more readable if this is returned by ogg_compute_chunk_table()
and passed where neccessary.

ogg_afh.c: make vi_sampling_rate, vi_bitrate local

ogg_afh.c: Kill unused vi_bitrate_nominal

ogg_afh.c: get rid of global variable max_chunk_len

It's only used in ogg_compute_chunk_table(). So make it a local variable.

simplify ogg_compute_chunk_table()

it rewinds the input file on entry and on exit. Do this in the caller instead
which allows to get rid of the file parameter in ogg_compute_chunk_table().

ogg_afh.c: Kill global variable infile

Pass it where neccessary instead.

ogg_afh.c: Kill superflous vi_channels

ogg_afh.c: Kill unused inbuf and inbuf_size

mp3_afh.c: Kill unused inbuf and inbuf_size;

simplify vss_read_chunk()

All its parameters are known globally within vss.c, so omit them.
Also, return an int instead of the pointer to the inbuffer which is
also known to the caller.

This patch also introduces para_fseek() in fd.c which is used by
mp3.c and vss.c and changes the mp3 audio format handler so that it
uses the new function and always check the return value.

kill the read_chunk handler of struct audio_format handler

chunks are now read by vss_read_chunk(), i.e. the same function is
used for all audio formats.

make get_file_info() return the chunk table

This is a preparation for the moving the read_chunk handler to
the vss layer.

vss.c: Kill check for get_file_info == NULL

This can never happen.

kill the reposition handler in struct audio_format_handler

All three audio format handlers provided only a dummy function. Kill
these as well.

unify all audio format handlers

This changes the ogg vorbis audio format handler such that its read_chunk and
reposition_stream functions coincide with those of the other audio format
handlers.

ogg_afh.c: Rename oggbuf_len to inbuf_size

so that it matches the naming of the other audio format handlers

ogg_afh.c: Rename oggbuf to inbuf

To make naming consistent with the aac and mp3 audio format handler.

unify aac and mp3 audio format handlers

This makes the mp3 audio format handler use a chunk table just like
m4a and ogg vorbis already do. Moreover, the aac and mp3 audio format
handlers now have identical read_chunk and reposition_stream handlers.

Next step is to tweak the ogg vorbis audio format handler so that
it also fits into this scheme. In the end, this allows to move
read_chunk() and reposition_stream() from the audio format handlers
one layer up to the virtual streaming system and to kill the function
pointers in struct audio_format.

The advantage of this change that it avoids duplicated/similar code, and
that new audio format handlers can added easier.

string.c para_realloc(): report requested size on realloc failures

ogg_afh.c: chunk_table is an array of type size_t

simplify ogg_compute_header_len()

dccp_recv.c, stdout.h: change year in GPL header to 2007

These two files have changed in 2007, so..

rename DB_READ, DB_WRITE to AFS_READ, AFS_WRITE

Merge /fml/ag-raetsch/home/maan/scm/paraslash_meins/paraslash/

FEATURES update

osx_write.c: Kill sleep() on buffer underruns

This was a leftover from the mpg123 patch for mac os which does
more harm than good. It could cause para_write/para_audiod to
block under certain circumstances.

NEWS update

stdout.h: Kill unused bufsize

filter.c: Register tasks in proper order

The order is important as it determines the order in which
the pre_select functions are going to be called by the scheduler.

In the previous code, the scheduler called stdin->pre_select _before_
filter_chain->pre_select() which gave rise to the following scenario
(note that there's no filter_chain->post_select()):

stdin->pre_select(): set stdin for reading
filter_chain->pre_select(): nothing to do
stdin->post_select(): read buffer
stdin->pre_select(): do _not_ set stdin for reading as stdin buffer is full

For para_filter, after the last there are no fds set which causes the
scheduler to use the default timeout of 1s. This had the effect that
decoding of aac files started only after a few seconds.

With the roles of interchanged tasks the scenario is as follows:

filter_chain->pre_select(): nothing to do
stdin->pre_select(): set stdin for reading
stdin->post_select(): read buffer
filter_chain->pre_select(): convert buffer
stdin->pre_select(): set stdin _again_ as stdin buffer is not full

Alternative fix might be to use filter_pre_select() also for post_select.

rename db.c to afs.c and db.h to afs.h

db.c find_audio_files(): use int rather than char for cwd_fd

Merge /home/maan/scm/paraslash_fml/paraslash

simplify get_atts()

remove pointless ret variable and clarify comment.

fix typo in dccp_writer

Again, a serious typo that causes the dccp writer to not error
out under certain circumstances.

simplify setup_default_wng()

The default writer is known at compile time, so replace the code
that computes it at runtime by a #define in configure.ac.

http_recv.c: Fix typo in http_recv_open()

Quite serious as it causes the http receiver not to error
out if get_host_info() fails.

new codename, reset version to git

the paraslash-0.2.15 release tarball

paraslash 0.2.15

Makefile.in: fix target web/misc

It depends on @PACKAGE_TARNAME@-@PACKAGE_VERSION@.tar.bz2 rather than on
paraslash-git.tar.bz2.

Makefile.in: only build and install man pages for supported binaries

This makes it possible again to install from the git repo even if not
all binaries can be build, e.g. Mac OS which always lacks para_fade
(reverts a858de7cf9437cf9a3569eb4bc297c5b0e768994).

Also, don't refuse to install and don't complain if $VARDIR
(/var/paraslash) can not be created, as this is non-fatal.

configure.ac: Detect os type, add -Wl,-bind_at_load to oggvorbis ld flags on Darwin.

This gets rid of zillions of warnings of the kind

ld: warning suggest use of -bind_at_load, as lazy binding may result in errors or different symbols being used

on Mac OS X (10.4.6)

rc4.h: add doxygen file comment

fix client args info memory leak

filter_chain.c, recv_common.c: Fix another memory leak

There was still one leak left: argv[0] gets strdupped, but never freed.

dccp_recv.c: Fix memory leak in dccp_recv_open()

Missing freeaddrinfo().

audiod.c: fix memory leak in audiod_status_dump()

missing free() for decoder flags. Quite serious: We leaked some bytes on each
status change.

filter_chain(): Fix memory leak in parse_filter_args

Same bug as in parse_receiver_args(): argv must be freed (see previous patch).

recv_common: fix memory leak and invalid free in error path

As split_args() returns pointers that point within the given string,
it is wrong to free the individual pieces. However, the array of
substrings must be freed. parse_receiver_args() got both of these
wrong :(

cosmetics

crypt.c: fix memory leak in para_decrypt_buffer()

The client side had more or less the same memory leak as the server side:
The rsa structure wasn't freed.

fix a memory leak in the RSA key handling

RSA keys must be freed by using openssl's RSA_free() rather than the
usual free(). This leak turns out to be more serious as the amount
of leaked memory increased by about 300 bytes every time para_server
reread the user list (i.e. the hup command was executed or para_server
received SIGHUP).

user_list.c: Fix a memory leak

get_rsa_key() already allocates space for the rsa key pointer. Moreover,
u->perms was set to zero twice. Kill one instance.

simplify lookup_user()

It's easier to just pass the name of the user to be looked up. This
fixes a memory leak btw. because u.name was strdupped and never freed.
That leak wasn't serious though as is happened in the child process.

list.h, ortp.h: Add missing documentation

mysql_selector.c: Fix mbox command

An additional space crept in.

para.h fix a typo

README: minor cleanups

add para.h to the files used by doxygen

move gui-related symbols from para.h to new gui.h

more doxygen docu, some small cleanups

para.h: Add documentation of PARA_MAX, PARA_MIN, PARA_ABS

kill STRINGSIZE macro

It's kind of silly. Just open code the desired value.

playlist_selector.c, random_selector.c: make update_audio_file() static

command_util.sh: Make it read the function prototype from the source file

This is much cleaner as the commands differ by gcc attribute usage,
e.g. __a_unused.  But it also means that in the source code

(a) the opening bracket for the function body must not be on
the same line (com_hist got this wrong),

(b) for all command handlers, the parameters must be called
fd, argc and argv,

(c) If a command handler uses __noreturn, that attribute must
be the first word, see audiod's com_term().

So change the few occasions where this was not the case.

compress.c, wav.c: add documentation

Merge /home/maan/scm/paraslash_fml/paraslash

add more missing documentation

don't set the initial_delay_barrier if time diff is unreliable

Merge /fml/ag-raetsch/home/maan/scm/paraslash_meins/paraslash/

Conflicts:

Makefile.in

audiod: Add missing documentation

alsa_write.c: Add missing documentation

audiod.c: Kill unused variable pcd

audiod.h: Kill unused struct audiod_task

The audiod task is an ordinary struct task.

fade.c: [trivial] whitespace cleanup

sdl_gui.ggo: Kill duplicate authorship info

This info is already provided by the HANDLE_VERSION_FLAG macro

gui.c: make command pipe a nonblocking fd

audiod.c: [trivial] move close_stat_pipe() closer to its callers

audiod: kill handle_signal()

It was only called from singal_event_handler(), so move the code there.
Also, remove the redundant check for t->ret != -E_SIGNAL_CAUGHT which
can never happen.

audiod: fix mode cycling

This was quite buggy all the time, but wasn't noticed due to the typo bug
which was fixed in 1b46ae4193cddb7d589b0f2f7d8158b84f7e9f6d.

Now, the status task no longer has an event handler as both its pre_select
and its post_select functions always succeed and the status task is supposed
to never terminate. Unlike the client task whose event handler now does all
the cleanup if the connection to para_server breaks down for whatever reason.

client_common.c: invalidate the private client data pointer on errors.

Not really neccessary, but clean.

use own version handler for all commands.

This makes the man pages generated by help2man look much nicer.

client: combine client_open() and client_parse_config()

Both users (client.c and audiod.c) called client_open() right
after calling client_parse_config(), so call client_open() from
client_parse_config() and make client_open() static.

While we're at it, rename client_open() to client_connect() as this
is what the function does. Also rename client_parse_config() to
client_open() and change audiod.c and client.c to call client_open()
instead of client_parse_config().

This patch fixes a segfault in para_client was it was run without
specifying a command. In this case, the private client data struct
was used by para_log after being freed -- Ouch. Fortunately, this
bug didn't affect para_audiod as it never calls these functions with
no command.

Another bug due to a typo in client_close() is also fixed. This code
needs closer investigation and lacks documentation btw.

para.h: Kill unused LICESE macro

Merge ../paraslash_meins/paraslash

fix typo in FEATURES

string.c: Add missing \return in documentation

new feature list, merge of README and FEATURES

README and FEATURES contained quite some overlap, so merge them
into a single document (README) and change FEATURES to contain only
a bullet list.

remove old server and audiod screenshots

update para_server and para_audiod "screenshots"

use current versions of the startup log messages.

overview.sk: Update to 0.2.15

audiod does not fork and exec anymore for quite some time now.

move s_a_r() and s_a_r_list() from string.c to mysql_selector.c

And make them static as they are only used in mysql_selector.c.

This reduces the stripped code size of all executables (except
para_server of course) by about 1K.

string.c: Documentation update

- Add documentation of return value to all functions.
- Replace \a by \p which looks nicer in the html output.
- Clarify s_a_r documentation

Merge ../paraslash_meins/paraslash

make man dirs before calling help2man and friends

Also, make sure we call help2man on the executables in the
current dir.

Merge ../paraslash.fml/paraslash

clarify documentation of mysql_selector_init()

mysql_selector.c: free(NULL) is OK.

No need to check for NULL. Moreover, stream, meta, atts and dir are
set in any case so kill the NULL initialization.

Makefile.in: minor cleanups

move target para_recv down and kill unneccessary -o option for the
mysql_selector.o target.

check_writer_arg(): change writer arg to const char *