paraslash 0.3.3

configure.ac: Remove unused variable filter_filters.

configure.ac: Remove amp_filter from set of executables.

It is not an executable.

oggdec_filter.ggo: Add more documentation.

oggdec_parse_config(): Add some sanity checks.

wav: Only create wav header if there is output available.

Due to recent changes to filter_chain.c, the convert functions of all
activated filters are called even if there is no output availabe since
the ogg vorbis decoder needs this.

However, this change broke the wav filter: wav_convert() is now called
even if the previous filter (e.g. oggdec) has not yet determined the
format of the input stream.

This caused the wav filter to write out an invalid wav header with 0
channels. Fix it by not doing anything unless wav_convert() gets called
with non-empty input.

para_server: Reformat help text of the --afs_database_dir option.

This caused help2man to create a badly formated man page.

Add -Wmissing-noreturn to compiler flags.

And mark all exit functions of gui.c with __noreturn.

Fix -Wdeclaration-after-statement warnings.

This option is new in gcc-4, so do not activate it by default.

Implement aac meta info support.

Avoid "no connection to para_server" message is server is stopped.

filter: Call the convert function even if the input buffer is empty.

At least the ogg vorbis decoder might still produce output in this
case.

error.h cleanups.

Merge commit 'fml/master'

afh_common.c: Add missing source code documentation.

Document afh_get_chunk() and make_taginfo().

write.h: Cosmetics.

oggdec: Keep decoding on imput EOF.

This bug causes the last chunk of the input not written out under
certain circumstances.

afh.c: Skip emtpy chunks.

Fix a gcc warning in alsa_open() on x86_64.

alsa_write.c:136: warning: format '%d' expects type 'int', but argument 4 has type 'size_t'

Get rid of a gcc warning on recent Ubuntu systems.

fchdir() is marked with warn_unused_result which produces a
warning in the error path of para_opendir(). However, we really
want to ignore the return value in this particular case, so introduce
a rather ugly fix to avoid the warning.

Check return value of write() in generic_signal_handler().

Abort if we could not write the signal number to the pipe (which
should never happen).

para_write: Fix --bufsize.

The given value means kilobytes, rather than bytes. Add checks
for invalid values of the buffer size.

alsa: Log info about the number of bytes per frame.

alsa: Improve chunk timings.

And get rid of magic constants.

para_write: Fix --initial-delay.

REQUIREMENTS: Fix dead link to faad lib.

README.afs: Minor improvments and clarifications.

write_common.h: Kill an unused function declaration.

Minor documentation improvements.

Better error diagnostics for the mp3 decoder.

http_recv: Fix reading of the HTTP OK message pattern.

This bug leads to sound artefacts at the beginning of the
stream caused by calling recv_pattern() with a buffer size
larger than the HTTP OK message.

It may thus happen that the buffer used by recv_pattern() contains
not only the HTTP OK message but also the beginning of the stream.
Since that buffer is discarded by recv_pattern(), the first few
bytes of the stream are lost if this happens.

Fix it by passing the exact length of the HTTP OK message to
recv_pattern().

Fix an off-by-one bug in recv_pattern().

The current code in recv_pattern() allocates a buffer of size
bufsize + 1 and calls recv_buffer() with bufsize as the size
parameter. However, recv_buffer() reserves the last byte of the
buffer for storing the terminating NULL byte, so that at most
bufsize - 1 characters are read.

Fix it by passing bufsize + 1 (the real size of the buffer)
to recv_buffer().

gui: Add the amplification value to output.

No need to check the return value of para_strdup().

REQUIREMENTS: Add the name of the debian package for faad.

REQUIREMENTS: Mention that libid3tag0-dev should  be installed for id3v2.

It's not enought to install only libid3tag0 as we need the header files
too.

Fix a use-without init buf in recv.c.

We need to init not only the receiver task but also the stdout task structure.

Fix the osx writer on little endian machines.

We have to use kLinearPCMFormatFlagIsBigEndian only of we're on a little endian
(intel) system.

call_callback(): Fix a shm leak in afs.c.

We forgot to detach the shm area passed from the child which results in
"no space left" messages if many audio files are added and the number
of shared memory areas exceeds 4096.

add_one_audio_file(): Fix an fd leak.

This bug bites if one tries to add more than 1024 new files with a single add
command.

guess_audio_format(): Return proper error code.

This could cause para_server to return bogus messages back to
the client, including messages containing the string "(null)".

INSTALL: Add comment that /var/paraslash should be writable.

Thanks to Johannes Wörner for pointing this out.

exec.c: Open /dev/null r/w.

Just to prevent external programs from getting confused when they write
to fd 1 or 2.

mp3_afh.c: Fix compilation on Mac OS 10.5.

uint is not defined on newer Mac OS X.

stdin: Abort if the consumer terminates.

The bug in the wav filter which was fixed in the previous patch
could cause para_filter to hang because the stdin task would not
care whether the filter task has terminated.

Fix it by introducing an output_error pointer.

Fix the wav filter for Mac OS 10.5.

We have to initialize fn->loaded.

REQUIREMENTS update.

Mention that the libmad development package is called libmad0-dev and
reformat the section on id3 tags so that grutatext indents it nicely.

Implement better error diagnostics for para_client.

In case the config file contains errors, the old code caused
para_client to exit without printing an error message.

We need print_errors == 0 in client_open() because this function
is also used by para_audiod which might not have a controlling
terminal. So use gengetopt's --no-handle-error option that instructs
the command line parser not to exit on errors. This wy we can use
PARA_ERROR_LOG() to log an appropriate error message.

amp_filter.c: Fix filename in doxygen comment.

Makefile.in: Do not use depricated git-archive.

Merge commit 'meins/master'

Fix com_init.

Dooh. The init command always produces output, so we have to pass a pointer
to send_result() as the result handler and the fd to send data to.

Thanks to Johannes Wörner for pointing out this rather embarrassing bug.

aft.c: Clear more status items in stop mode.

vss: Fix com_jmp() and com_ff().

We must not clear the VSS_REPOS flag in vss_send_chunk() as this
might cause the reposition request being ignored.

Clear all fields in stop mode where it makes sense.

Merge commit 'meins/master'

NEWS update.

Fix a bug in para_write.

With the old code, the command "para_write < /dev/null" would
segfault because the wng_close() is called although the writer
node group was never opened. Fix this bug by introducing a new
field in struct writer_node_group that tracks whether wng_open()
was called.

Add the new amp filter.

Move the amplification code from compress.c to amp_filter.c

Merge commit 'meins/master'

fade: Fix default modes.

Merge commit 'meins/master'

score_update(): Don't return an error if file is not admissible.

This function is called whenever the afs_info struct changed,
for example if the user executed a touch command. Simply
ignore the request to update the score in case the file is not
contained in the score table.

Print more info in case an afs event handler failed.

Fix another bug in com_touch().

Only set the amp value if the -a option was given.

compress filter: Fix parsing of the amplification string.

We need to consider the prefix as well.

Fix com_touch.

The check whether no options were given was busted.

NEWS update.

Add more documentation about the amplification value.

compress.c: Activate pre-amplification.

Use the amplification value announced by para_server to
pre-amplify the stream.

Initialize stat_item_values[].

Just to be sure..

audiod: Make stat_task static.

The only reason it wasn't static was that audiod_command.c
could use it. However, audiod_command.c only needs
stat_item_values[] and several other fields that were only
used in get_time_string().

So move stat_item_values[] out of struct status task so that
audiod_command.c can use it, and move get_time_string() from
audio_command.c to audiod.c.

Teach the touch command about amplification.

Teach the ls command to print the amplification value.

Use an unused byte in struct afs_info to store the amplification value.

Introduce new status item: amplify.

Merge commit 'meins/master'

configure.ac: Print whether libid3tag was found.

REQUIREMENTS: Add a note about libid3tag.

Implement ogg vorbis comment parsing.

Move make_taginfo() from mp3_afh.c to afh_common.c.

This way, also the ogg vorbis code can use it.

mp3: Add support for id3 version 2 tags.

Keep the audio file open when mmapping.

libid3tag needs an open fd. So call () with a
non-NULL pointer which results in the file still being open
after mmap_full_file() returns. Pass the file descriptor
to compute_afhi() and to get_file_info() so that libid3tag
can use it.

Link para_server and para_afh against libid3tag if available.

Make mp3_get_file_info() static.

Reorder functions in mp3_afh.c.

Get rid of struct id3tag and struct mp3info.

As a preparation for dealing with id3v2 tags.

mp3_read_info(): Kill fl_avg.

The average frame length is computed but not used anywhere.

Kill struct mp3info->header.

Make mp3_seek_next_header() more readable.

It fills in a mp3header structure. So use a function parameter
instead of a global variable.

Simplify mp3_seek_next_header().

Allocate afh_info->info_string dynamically.

Merge commit 'meins/master'

mp3_afh.c: Simplify mp3_get_id3().

No need to pass the fpos pointer here.

osl.c: Do not include list.h.

[NET]: Bug-fix for getaddrinfo()

Something changed in the getaddrinfo() mechanism, causing AF_UNSPEC
in combination with ai_socktype=0 to return IPv4 addresses first
(instead of starting with IPv6 addresses). As a consequence, IPv6
servers are restricted to only IPv4 connections - IPv6 clients will
receive an ICMPv6 error message, which gets translated into an annoying
"protocol error" locally.

The fix is to pretend to be UDP instead of DCCP (pretending to be TCP
also works). This allows getaddrinfo to look up the address without
side effects and has been found to work.

recv.h: Cosmetics.

fade.c: Stop playing if no sleep mode was given.

stat.c: Comment out noisy debug messages.

filter_chain.c: Cosmetics.