paraslash.git
8 years agogcrypt: Remove pointless assignment.
Andre Noll [Tue, 15 May 2012 20:16:38 +0000 (22:16 +0200)]
gcrypt: Remove pointless assignment.

gcrypt.c:700:6: warning: Assigned value is always the same as the existing value
        ret = key->num_bytes;
        ~~~ ^ ~~~~~~~~~~~~~~

8 years agogcrypt: Remove dead stores.
Andre Noll [Tue, 15 May 2012 20:15:41 +0000 (22:15 +0200)]
gcrypt: Remove dead stores.

gcrypt.c:502:2: warning: Value stored to 'cp' is never read
        cp += ret;
        ^     ~~~
gcrypt.c:577:2: warning: Value stored to 'cp' is never read
        cp += ret;
        ^     ~~~

8 years agomp3dec: Remove dead store.
Andre Noll [Tue, 15 May 2012 20:13:09 +0000 (22:13 +0200)]
mp3dec: Remove dead store.

mp3dec_filter.c:133:3: warning: Value stored to 'ret' is never read
                ret = mad_stream_sync(&pmd->stream);
                ^     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

8 years agoMakefile: Warn for unused macros only on Linux.
Andre Noll [Mon, 4 Jun 2012 08:29:42 +0000 (10:29 +0200)]
Makefile: Warn for unused macros only on Linux.

We need these macros at several places, but each of them causes
a compilier warning on FreeBSD and NetBSD where these macros are
not used at all.

This patch moves the -Wunused-macros option to the Linux-only
section.

8 years agoaudiod: Check pointers passed to btr_get_node_start().
Andre Noll [Sat, 7 Apr 2012 22:25:42 +0000 (00:25 +0200)]
audiod: Check pointers passed to btr_get_node_start().

If the receiver or writer buffer tree node pointer is NULL, we'll segfault due
to a a NULL pointer dereference in btr_get_node_start(). Although this might
not be triggerable, it does not hurt to be defensive here.

8 years agooss: Add proper error message on init.
Andre Noll [Wed, 16 May 2012 13:21:10 +0000 (15:21 +0200)]
oss: Add proper error message on init.

If the device node of the sound device (/dev/dsp) does not exist,
we currently fail with a "no such file or directory" error message
without mentioning the name of the device. This adds a log message
which clarifies what went wrong.

8 years agoMerge branch 't/remove_no_default_filters'
Andre Noll [Sat, 5 May 2012 10:50:37 +0000 (12:50 +0200)]
Merge branch 't/remove_no_default_filters'

We have printed a warning for several month if this option was
still used. This should be enough.

8 years agoRemove --no_default_filters option.
Andre Noll [Sun, 24 Jun 2012 17:26:16 +0000 (19:26 +0200)]
Remove --no_default_filters option.

This option was deprecated in paraslash-0.4.10 and can go away now.

8 years agoMerge branch 't/doc'
Andre Noll [Sat, 5 May 2012 10:48:27 +0000 (12:48 +0200)]
Merge branch 't/doc'

8 years agodaemon.c: daemon_set_default_log_colors() returns void.
Andre Noll [Tue, 24 Apr 2012 06:43:32 +0000 (08:43 +0200)]
daemon.c: daemon_set_default_log_colors() returns void.

Remove the incorrect documentation of the (non existing) return
value.

8 years agoClient: Simplify signal handling.
Andre Noll [Mon, 9 Apr 2012 19:07:23 +0000 (21:07 +0200)]
Client: Simplify signal handling.

There is no point in having a sighandler which just calls another one.
This patch makes i9e_signal_dispatch() the handler for SIGINT and
removes the pointless wrapper.

8 years agorecv.h: Fix typo in documentation.
Andre Noll [Sat, 14 Apr 2012 22:04:29 +0000 (00:04 +0200)]
recv.h: Fix typo in documentation.

The sentence contains the word "the" twice.

8 years agoFix a typo in the documentation of struct ggo_help.
Andre Noll [Sat, 14 Apr 2012 21:59:52 +0000 (23:59 +0200)]
Fix a typo in the documentation of struct ggo_help.

s/exexutables/executables.

8 years agoAdd some more starting points to main page.
Andre Noll [Sat, 14 Apr 2012 21:28:57 +0000 (23:28 +0200)]
Add some more starting points to main page.

This adds links to struct writer_node, the flacdec filter and to
interactive.c to the main doxygen web page. It also fixes a typo.

8 years agoAdd some missing documentation.
Andre Noll [Sat, 14 Apr 2012 21:17:03 +0000 (23:17 +0200)]
Add some missing documentation.

This makes doxygen a bit happier.

8 years agobtr_slice_out_node(): Set parent to NULL.
Andre Noll [Sat, 7 Apr 2012 22:30:23 +0000 (00:30 +0200)]
btr_slice_out_node(): Set parent to NULL.

Without this, a subsequent btr_remove_node() on the spliced
out node could segfault.

8 years agointeractive: Init task status.
Andre Noll [Sat, 7 Apr 2012 18:04:05 +0000 (20:04 +0200)]
interactive: Init task status.

The log messages of the scheduler print the task status, so all
tasks should fill out the task.status field. The interactive task
missed to to so. Fix is trivial.

8 years agoIntroduce clear_afhi().
Andre Noll [Tue, 23 Aug 2011 17:41:27 +0000 (19:41 +0200)]
Introduce clear_afhi().

aft.c and afh.c contain identical code to free all members of an
instance of struct afh_info. This commit introduces the public
function clear_afhi(), and patches both aft.c and afh.c to call the
new function.

8 years agohttp_recv: Trivial cleanup.
Andre Noll [Wed, 24 Aug 2011 12:38:39 +0000 (14:38 +0200)]
http_recv: Trivial cleanup.

This variant to specify the size of an allocation is to be preferred
because (a) it's shorter, and (b) it breaks at compile time if "tmp" is
renamed but the argument to sizeof() was not updated accordingly.

8 years agoDefine status item list in afh_common.c.
Andre Noll [Thu, 5 Apr 2012 23:18:23 +0000 (01:18 +0200)]
Define status item list in afh_common.c.

The status_item_list array is needed for the audio format handler info
struct (afhi), so all executables which contain afh_common.o need it.
Currently, afh.c and command.c define their own instance for para_afh
and para_server respectively.

Moving the definition of the array to afc.c saves one instance. No
big deal.

8 years agobtr_remove_node(): Loglevel adjustments.
Andre Noll [Thu, 5 Apr 2012 23:18:23 +0000 (01:18 +0200)]
btr_remove_node(): Loglevel adjustments.

These two log messages got a bit too noisy. Degrade loglevel to info.

8 years agointeractive: Set stderr to nonbuffered mode.
Andre Noll [Thu, 5 Apr 2012 23:18:23 +0000 (01:18 +0200)]
interactive: Set stderr to nonbuffered mode.

fdopen() might return a stream which is fully buffered or line buffered.
We want unbuffered unconditionally, so explicitly set the stream to
unbuffered mode.

8 years agointeractive: Remove pointless return statement.
Andre Noll [Thu, 5 Apr 2012 23:18:23 +0000 (01:18 +0200)]
interactive: Remove pointless return statement.

i9e_pre_select() returns void anyway.

8 years agonew codename, reset version to git
Andre Noll [Fri, 30 Mar 2012 14:18:14 +0000 (16:18 +0200)]
new codename, reset version to git

8 years agothe paraslash-0.4.10 release tarball
Andre Noll [Fri, 30 Mar 2012 14:10:28 +0000 (16:10 +0200)]
the paraslash-0.4.10 release tarball

8 years agoparaslash 0.4.10 v0.4.10
Andre Noll [Fri, 30 Mar 2012 14:07:52 +0000 (16:07 +0200)]
paraslash 0.4.10

8 years agoMerge branch 't/gui_improvements'
Andre Noll [Thu, 29 Mar 2012 17:58:04 +0000 (19:58 +0200)]
Merge branch 't/gui_improvements'

      gui: Make some functions return void.
      gui: Rename client_cmd_cmdline() to para_cmd().
      gui: Check return value of para_exec_cmdline_pid().
      gui: Also display command's stderr output.

8 years agoMerge branch 't/configure_fix'
Andre Noll [Sun, 25 Mar 2012 18:32:23 +0000 (20:32 +0200)]
Merge branch 't/configure_fix'

      configure: Fix warning message.

8 years agoMerge branch 't/id3_conf'
Andre Noll [Sun, 25 Mar 2012 15:53:07 +0000 (17:53 +0200)]
Merge branch 't/id3_conf'

configure: Add --with-id3tag-headers/libs.
Link against libz if compiling with libid3tag support.

8 years agomood.c: Silence gcc warning.
Andre Noll [Sun, 25 Mar 2012 12:51:25 +0000 (14:51 +0200)]
mood.c: Silence gcc warning.

The newly released gcc-4.7.0 complains about "m" being used
uninitialized in change_current_mood(). gcc is wrong, but this is
not obvious to see: In change_current_mood(), "m" is only going to
be used if load_mood() returns non-negative. This happens only on
success, when load_mood() returns 1. In this case "m" was previously
set to mlpd.m which was initialized to NULL at the top of load_mood()
and later set to the newly allocated mood structure.

This patch makes it easier for gcc by initializing "m" to NULL
upfront. This causes the warning to go away.

8 years agoLink against libz if compiling with libid3tag support.
Andre Noll [Tue, 13 Mar 2012 15:37:36 +0000 (16:37 +0100)]
Link against libz if compiling with libid3tag support.

libid3tag depends on libz. On Linux, libz is automatically linked to
para_server and para_afh but at least on FreeBSD compilation fails
due to undefined references to "uncompress" and "compress2".

This patch explicitly adds -lz to $server_ld_flags and $afh_ldflags
if libid3tag was detected.

8 years agoconfigure: Add --with-id3tag-headers/libs.
Andre Noll [Wed, 7 Mar 2012 17:16:52 +0000 (18:16 +0100)]
configure: Add --with-id3tag-headers/libs.

If libid3tag was intalled at a non-standard location, the configure
script would not find it. Currently there is no way to tell configure
to look for libid3tag elsewhere.

This adds two new options, --with-id3tag-headers and --with-id3tag-libs,
to overcome this shortcoming.

8 years agogui: Also display command's stderr output.
Andre Noll [Tue, 13 Mar 2012 21:17:18 +0000 (22:17 +0100)]
gui: Also display command's stderr output.

Currently, both para_cmd() and display_cmd() redirect stderr to
/dev/null, so any error messages of the command being executed do
not make it to the bottom window. This is inconvenient at times,
and makes debugging hard.

This patch prints the stderr output in the bottom window using
COLOR_ERRMSG.

As for the implementation, the global command_pipe integer
variable becomes an integer array of length two which contains
the two file descriptors for stdout and stderr. We also need two
buffers in do_select(), so command_buf becomes a two-dimensional
array. do_select() now has to monitor two file descriptors per command
and it returns only after the peer has closed both fds.

8 years agogui: Check return value of para_exec_cmdline_pid().
Andre Noll [Tue, 13 Mar 2012 21:30:34 +0000 (22:30 +0100)]
gui: Check return value of para_exec_cmdline_pid().

external_cmd() calls this function to execute the given command.
However, as para_exec_cmdline_pid() may fail, e.g. because the
underlying call to fork() failed due to hitting the RLIMIT_NPROC
resource limit, we must check the return value. The other two command
dispatchers got this right, but external_cmd() did not.

8 years agogui: Rename client_cmd_cmdline() to para_cmd().
Andre Noll [Tue, 13 Mar 2012 21:19:25 +0000 (22:19 +0100)]
gui: Rename client_cmd_cmdline() to para_cmd().

The new name is shorter to type and it matches the naming convention
of the other command dispatchers, external_cmd() and display_cmd().

8 years agogui: Make some functions return void.
Andre Noll [Tue, 13 Mar 2012 20:11:16 +0000 (21:11 +0100)]
gui: Make some functions return void.

All callers of these functions ignore the return value, so get rid
of it.

8 years agoconfigure: Fix warning message.
Andre Noll [Wed, 7 Mar 2012 16:01:05 +0000 (17:01 +0100)]
configure: Fix warning message.

If libogg is installed but speex is not, configure prints "vorbis/speex
require ogg", which is a rather bad explanation of the problem at hand.

This simple patch should fix it.

8 years agoMerge branch 't/t0004_fix'
Andre Noll [Sun, 18 Mar 2012 20:02:33 +0000 (21:02 +0100)]
Merge branch 't/t0004_fix'

This topic has been cooking for some weeks now and was tested
successfully on all supported operating systems.

8 years agomanual: Explain how to compile from git.
Andre Noll [Thu, 15 Mar 2012 12:01:03 +0000 (13:01 +0100)]
manual: Explain how to compile from git.

Mention that autogen.sh should be executed first when compiling from git or
gitweb snapshots.

8 years agodoc: Avoid \0 in comments.
Andre Noll [Wed, 14 Mar 2012 13:41:50 +0000 (14:41 +0100)]
doc: Avoid \0 in comments.

Doxygen treats \0 as a command, spits out a warning and produces
no output.

8 years agoMerge branch 't/audioc_cleanups'
Andre Noll [Sun, 4 Mar 2012 21:40:27 +0000 (22:40 +0100)]
Merge branch 't/audioc_cleanups'

Has been cooking for long enough.

8 years agoFix t0004-server.
Andre Noll [Sun, 5 Feb 2012 13:39:37 +0000 (14:39 +0100)]
Fix t0004-server.

Currently we try to generate an ssh key and start para_server
even if the ssh-keygen executable was not found or para_server
was not built.

Fix this by starting para_server only of both executables exist.

8 years agoconfigure.ac: Do not include server-only objects unconditionally.
Andre Noll [Sun, 5 Feb 2012 13:18:11 +0000 (14:18 +0100)]
configure.ac: Do not include server-only objects unconditionally.

We included the list of server-only objects in $all_errlist_objs
even if para_server is not going to be built (because libosl was
not found). This confuses t0004-server which bases the decision
whether it should skip its tests on whether server.o is contained
in $all_errlist_objs.

So add these objects to $all_errlist_objs only if libosl is detected.

8 years agoconfigure.ac: Remove fec from objects for para_recv.
Andre Noll [Sun, 5 Feb 2012 13:10:05 +0000 (14:10 +0100)]
configure.ac: Remove fec from objects for para_recv.

para_recv does not need this object.

8 years agoconfigure.ac: Kill $gui_other_objs.
Andre Noll [Sun, 5 Feb 2012 12:36:49 +0000 (13:36 +0100)]
configure.ac: Kill $gui_other_objs.

Just treat these two objects as ordinary objects which do
not define own error codes.

8 years agoconfigure.ac: Try to link against libogg and lm if necessary.
Andre Noll [Mon, 27 Feb 2012 12:44:26 +0000 (13:44 +0100)]
configure.ac: Try to link against libogg and lm if necessary.

This seems to be necessary on FreeBSD, and does hopefully not hurt
on other systems.

8 years agoMerge branch 't/mmap_sanity'
Andre Noll [Sun, 26 Feb 2012 13:23:59 +0000 (14:23 +0100)]
Merge branch 't/mmap_sanity'

Conflicts:
error.h

8 years agoMerge branch 't/write_consolidation'
Andre Noll [Sat, 18 Feb 2012 12:21:38 +0000 (13:21 +0100)]
Merge branch 't/write_consolidation'

This topic branch has been cooking in next for two weeks with no
problems.

8 years agoaudioc: Improve documentation of main().
Andre Noll [Fri, 13 Jan 2012 22:59:12 +0000 (23:59 +0100)]
audioc: Improve documentation of main().

8 years agoMerge branch 't/deprecate_no_default_filters'
Andre Noll [Sun, 12 Feb 2012 03:14:57 +0000 (04:14 +0100)]
Merge branch 't/deprecate_no_default_filters'

8 years agoaudioc: Fix memory leak in configfile_exists().
Andre Noll [Fri, 13 Jan 2012 22:54:08 +0000 (23:54 +0100)]
audioc: Fix memory leak in configfile_exists().

In case the config file does not exist, the function returns NULL
without freeing the config_file buffer.

This patch also makes config_file non-static. As configfile_exists()
is called at most once, it is pointless to have a static variable
there. The condition "if (!config_file)" is always true, so we can
get rid of the conditional alltogether.

8 years agoaudioc: Do not leak command buffer.
Andre Noll [Fri, 13 Jan 2012 22:43:05 +0000 (23:43 +0100)]
audioc: Do not leak command buffer.

This buffer contains the command to be sent to para_audiod. It is
allocated but never freed.

8 years agoaudioc.c: Always free IO buffer.
Andre Noll [Fri, 13 Jan 2012 22:39:56 +0000 (23:39 +0100)]
audioc.c: Always free IO buffer.

This 8K buffer is allocated but never freed.

8 years agoFix memory leak in audioc.c.
Andre Noll [Fri, 13 Jan 2012 22:38:14 +0000 (23:38 +0100)]
Fix memory leak in audioc.c.

We leak cf which is allocated and returned by configfile_exists().

8 years agoImprove error diagnostics of mmap_full_file().
Andre Noll [Thu, 8 Dec 2011 20:01:01 +0000 (21:01 +0100)]
Improve error diagnostics of mmap_full_file().

On mmap errors, this function used to return the error code it got
from mmap(). Unfortunately, in at least two cases, this results in
rather obscure error messages:

$ para_afh -i .
main: No such device
$ touch foo; para_afh -i foo
main: Invalid argument

This simple patch specifically checks whether the given fd refers
to a directory or an empty file so that the error messages become
more readable:

$ para_afh -i .
main: failed to mmap "."
main: Is a directory
$ touch foo; para_afh -i foo
main: failed to mmap "foo"
main: file is empty

8 years agograb_client.c: Fix another documentation bug
Andre Noll [Mon, 30 Jan 2012 16:26:23 +0000 (17:26 +0100)]
grab_client.c: Fix another documentation bug

In gc_activate(), the pointer to the scheduler struct is called "s"
rather than "sched".

8 years agoMerge commit '1678ee'
Andre Noll [Mon, 30 Jan 2012 07:30:40 +0000 (08:30 +0100)]
Merge commit '1678ee'

8 years agoaudiod: Deprecate the --no_default_filters option.
Andre Noll [Thu, 1 Dec 2011 18:25:47 +0000 (19:25 +0100)]
audiod: Deprecate the --no_default_filters option.

This option does not make much sense anymore these days as we have
sane defaults in case no filter was given. We prefer deprecation over
removal because this allows to switch between old and new versions
without having to edit the config file.

The option will be removed after the next release.

8 years agopara_client: Fix a memory leak.
Andre Noll [Sun, 22 Jan 2012 21:45:21 +0000 (22:45 +0100)]
para_client: Fix a memory leak.

The main function of para_client first calls client_parse_config()
to decide whether to start an interactive session, run in completion
mode or execute the given command. In the third case case we call
client_open() which calls client_parse_config() again, thereby
leaking the client_task structure which was returned by the first
call to client_parse_config().

Fix this leak by calling client_connect() instead of client_open().

8 years agoFix a typo in the manual.
Andre Noll [Sat, 21 Jan 2012 13:53:32 +0000 (14:53 +0100)]
Fix a typo in the manual.

8 years agoReplace PARA_VSNPRINTF by xvasprintf().
Andre Noll [Fri, 6 Jan 2012 03:15:34 +0000 (04:15 +0100)]
Replace PARA_VSNPRINTF by xvasprintf().

The PARA_VSNPRINTF macro is rather clumsy, and too large to be inlined.
Moreover, it does not return the length of the formated string, so
users have to call strlen() after the call to PARA_VSNPRINTF(). This
is extra work which can easily be avoided since the number of bytes
written is returned by the underlying call to vsnprintf().

This patch replaces the macro by the public function xvasprintf(),
which is similar to the non-standard vasprintf() on GNU systems. It
also adds xasprintf(), a similar variant which takes a variable number
of arguments.

Unlike PARA_VSNPRINTF, xasprintf() and xvasprintf() return the number
of bytes written. This relies on vsnprintf() conforming to the C99
standard and breaks in particular on glibc 2.0 systems. Since glibc
2.0 is about 15 years old, this is unlikely to cause problems on
real systems.

All users which called strlen() right after xvasprintf() are changed
to use the return value of xvasprintf() instead.

8 years agocommand: Introduce struct command_context.
Andre Noll [Sat, 24 Dec 2011 00:22:58 +0000 (01:22 +0100)]
command: Introduce struct command_context.

This exposes a couple of variables which were previously local to
handle_connect() to the command handlers, allowing them to make use
of the full command context.

All newly exposed variables are stored in an instance of the new
structure, and the command handlers now take a pointer to such
a structure.

The patch is large but also straight forward.

8 years agoIntroduce xwritev().
Andre Noll [Fri, 23 Dec 2011 17:28:45 +0000 (18:28 +0100)]
Introduce xwritev().

For the sideband API we will need to write two buffers one after
another. This patch adds the new public function xwritev() to fd.c
which takes an arbitrary number of buffers and calls writev() to
perform the write.

With this function in place, xwrite() becomes a trivial wrapper
for xwritev().

8 years agoRename write_nonblock() to xwrite().
Andre Noll [Tue, 20 Dec 2011 16:37:28 +0000 (17:37 +0100)]
Rename write_nonblock() to xwrite().

This function is not only useful for non-blocking file descriptors,
so the name was misleading. Rename it to xwrite() for the lack of
a better name.

8 years agosc_send_buffer(): Return an error on short writes.
Andre Noll [Sun, 11 Dec 2011 19:28:31 +0000 (20:28 +0100)]
sc_send_buffer(): Return an error on short writes.

This allows all callers to just check for negative return values.

8 years agosc_send_bin_buffer: Use write_nonblock() rather than write_all().
Andre Noll [Sun, 11 Dec 2011 19:27:27 +0000 (20:27 +0100)]
sc_send_bin_buffer: Use write_nonblock() rather than write_all().

The write_nonblock() variant is safer here as it allows to let the
callers deal with short writes.

Despite the name, write_nonblock() can be used for both blocking and
non-blocking fds. It will be renamed in a subsequent patch.

8 years agosc_send_result(): Error out on short writes.
Andre Noll [Sun, 11 Dec 2011 19:23:14 +0000 (20:23 +0100)]
sc_send_result(): Error out on short writes.

This affects only afs command handlers, where short writes should
never happen as the socket fd is set to blocking mode. But still.

8 years agoafh: Use write_all rather than plain write().
Andre Noll [Sun, 11 Dec 2011 19:21:53 +0000 (20:21 +0100)]
afh: Use write_all rather than plain write().

We are currently using plain write() to write out the audio
file header. If this results in a short write, or if the write
is interrupted, we should just try again. So it's safer to call
write_all() here.

8 years agofd: Let write_all() return an error on short writes.
Andre Noll [Sun, 11 Dec 2011 20:12:13 +0000 (21:12 +0100)]
fd: Let write_all() return an error on short writes.

Short writes are often ignored because most callers of write_all()
do not check the return value. Rather than fixing all callers, this
patch teaches write_all() to detect short writes and return the new
E_SHORT_WRITE error code in this case.

Currently write_all() is a wrapper for write(), similar to
write_nonblock() but lacking the safety checks of the latter. To get
these safety checks, write_all() is changed to call write_nonblock().

Unfortunately, this involves some code movement.

8 years agoFix two minor documentation issues.
Andre Noll [Thu, 19 Jan 2012 21:37:24 +0000 (22:37 +0100)]
Fix two minor documentation issues.

8 years agofd: Improve error handling of write_nonblock().
Andre Noll [Sun, 11 Dec 2011 18:48:56 +0000 (19:48 +0100)]
fd: Improve error handling of write_nonblock().

This function had two shortcomings: First, a call to write might
fail with errno set to EINTR in case the write call was interrupted
before any data was written. This is not fatal and one should just
retry the write in this case.

Secondly, POSIX allows to return either EAGAIN or EWOULDBLOCK if the
write would block but we only check for EAGAIN. This is no problem on
Linux since both constants refer to the same value on Linux. However,
POSIX does not require them to be equal, so we have to check for both.

This patch corrects both issues.

8 years agoMove send_buffer() and send_va_buffer() from net.c to fd.c.
Andre Noll [Tue, 6 Dec 2011 20:07:56 +0000 (21:07 +0100)]
Move send_buffer() and send_va_buffer() from net.c to fd.c.

These functions end up calling plain write(), hence they work on
arbitrary file descriptors, not just network sockets. So they really
belong to fd.c rather than to net.c. Rename the two functions to
write_buffer() and write_va_buffer().

8 years agoRemove send_bin_buffer().
Andre Noll [Tue, 6 Dec 2011 19:50:29 +0000 (20:50 +0100)]
Remove send_bin_buffer().

It is just a trivial wrapper for write_all(). This patch changes all
callers to use write_all() directly and removes the wrapper.

8 years agofd: Make write_all() receive the length by value.
Andre Noll [Tue, 6 Dec 2011 19:46:17 +0000 (20:46 +0100)]
fd: Make write_all() receive the length by value.

Not a single caller actually checked the value of the passed
len pointer after the call, which is a sure sign for a bad API.
Just return the number of bytes written.

8 years agolsatt: Do not print "no matches" message unconditionally.
Andre Noll [Fri, 20 Jan 2012 21:42:39 +0000 (22:42 +0100)]
lsatt: Do not print "no matches" message unconditionally.

com_lsatt() needs to know if output has been sent to the client in
order to decide whether it should print the "no matches" error message.
Currently this message is printed if at least one argument was given
and send_callback_request() returned zero.

However, send_callback_request() returns zero on success and negative
on errors. Hence, if at least one argument was given, the error
message is printed regardless of whether there was a match.

Fix this bug by changing send_callback_request() to return the number
of received shared memory areas on success.

8 years agocommand.c: Fix typo in comment.
Andre Noll [Sun, 15 Jan 2012 16:13:33 +0000 (17:13 +0100)]
command.c: Fix typo in comment.

8 years agoFix typo in audiod man page.
Andre Noll [Tue, 10 Jan 2012 18:56:39 +0000 (19:56 +0100)]
Fix typo in audiod man page.

8 years agoMerge branch 't/web_fix'
Andre Noll [Tue, 10 Jan 2012 18:43:25 +0000 (19:43 +0100)]
Merge branch 't/web_fix'

8 years agoChange year in copyright message to 2012.
Andre Noll [Sat, 7 Jan 2012 01:34:18 +0000 (02:34 +0100)]
Change year in copyright message to 2012.

8 years agoMake web pages conform to the HTML standard again.
Andre Noll [Fri, 6 Jan 2012 06:56:56 +0000 (07:56 +0100)]
Make web pages conform to the HTML standard again.

Without this fix, the w3c validator complains about end tags for
element <p> which is not open. Removing both start and end tags makes
it happy again and causes no visible change, at least in one browser.

8 years agoafh/audioc: Explicitly mark STDOUT as blocking.
Andre Noll [Tue, 6 Dec 2011 17:17:57 +0000 (18:17 +0100)]
afh/audioc: Explicitly mark STDOUT as blocking.

STDOUT_FILENO is usually set to blocking mode, but as we rely on this,
it is safer to clear the nonblock flag explicitly.

8 years agocom_init(): Return proper error code.
Andre Noll [Sun, 11 Dec 2011 19:24:53 +0000 (20:24 +0100)]
com_init(): Return proper error code.

If an error occurred during database initialization, we send an error
message to the client.  However, it this fails, we currently log
the send error rather than the error that caused that initialization
to fail. Fix this by logging the root cause of the problem instead.

8 years agoaudiod: Fix bug in status_pre_select().
Andre Noll [Fri, 23 Dec 2011 20:51:00 +0000 (21:51 +0100)]
audiod: Fix bug in status_pre_select().

We have to take the minimum input size into account when deciding
whether a minimal delay should be requested from the scheduler.
Without this, a busy loop is possible.

8 years agoMerge remote-tracking branch 's/master'
Andre Noll [Sat, 24 Dec 2011 17:30:07 +0000 (18:30 +0100)]
Merge remote-tracking branch 's/master'

8 years agosend: Make ->send_fec() return void.
Andre Noll [Mon, 5 Dec 2011 20:04:02 +0000 (21:04 +0100)]
send: Make ->send_fec() return void.

There is only one caller, vss_send(), which ignores the return value.
So there is no point in returning an error code from this method.

8 years agoAlways include stdbool.h.
Andre Noll [Sat, 3 Dec 2011 16:38:11 +0000 (17:38 +0100)]
Always include stdbool.h.

This adds the #include statement for stdbool.h to para.h. This allows
to get rid of the individual includes in *.c.

More importantly, since all *.c files include para.h, booleans will
now be available everywhere so that we won't need to touch dozens of
files anymore whenever a boolean is added to a public structure.

8 years agoweb: Fix year in release announcement.
Andre Noll [Tue, 6 Dec 2011 18:55:13 +0000 (19:55 +0100)]
web: Fix year in release announcement.

8 years agonew codename, reset version to git
Andre Noll [Tue, 6 Dec 2011 16:37:35 +0000 (17:37 +0100)]
new codename, reset version to git

8 years agothe paraslash-0.4.9 release tarball
Andre Noll [Tue, 6 Dec 2011 16:33:10 +0000 (17:33 +0100)]
the paraslash-0.4.9 release tarball

8 years agoparaslash 0.4.9 v0.4.9
Andre Noll [Tue, 6 Dec 2011 16:29:16 +0000 (17:29 +0100)]
paraslash 0.4.9

8 years agoMerge branch 't/interactive'
Andre Noll [Sun, 4 Dec 2011 23:13:32 +0000 (00:13 +0100)]
Merge branch 't/interactive'

Conflicts:
string.h

8 years agoMerge branch 't/flac'
Andre Noll [Tue, 29 Nov 2011 07:36:49 +0000 (08:36 +0100)]
Merge branch 't/flac'

8 years agoMerge branch 't/generic_match_count'
Andre Noll [Fri, 25 Nov 2011 12:13:35 +0000 (13:13 +0100)]
Merge branch 't/generic_match_count'

8 years agoDocument interactive sessions and add link to libreadline.
Andre Noll [Fri, 9 Sep 2011 16:29:25 +0000 (18:29 +0200)]
Document interactive sessions and add link to libreadline.

8 years agoRewrite bash completion.
Andre Noll [Thu, 8 Sep 2011 23:58:35 +0000 (01:58 +0200)]
Rewrite bash completion.

Now that para_client and para_audioc gained the --completion option for
completing a partially entered command line, we can use this feature
in the bash completion script. This allows to avoid duplicating the
available commands in the completion script.

8 years agoInteractive mode for para_audioc.
Andre Noll [Mon, 5 Sep 2011 18:46:06 +0000 (20:46 +0200)]
Interactive mode for para_audioc.

This is in the same spirit as the previous commit which added
interactive session support for para_client. It implements command
completion and command line history for para_audioc.

Just as for para_client the new completion code in audioc.c is
compiled in only if libreadline was found by configure. In this case
para_audioc starts an interactive session if no command was given at
the command line.

8 years agoclient: Implement interactive mode.
Andre Noll [Mon, 5 Sep 2011 13:47:34 +0000 (15:47 +0200)]
client: Implement interactive mode.

This makes para_client enter an interactive session when started with no
command. Command line history and command completion are available in
interactive sessions.

This populates the previously empty files interactive.h and interactive.c which
contain the readline/interactive specific part. Everything in these files is
independent of para_client.  Conversely, client.c and client_common.c are
independent of readline.

The public API defined in interactive.h was designed to be reused from other
applications. In fact, a subsequent commit changes para_audioc to offer
interactive sessions as well.

8 years agostring: Introduce compute_word_num().
Andre Noll [Sun, 28 Aug 2011 21:36:17 +0000 (23:36 +0200)]
string: Introduce compute_word_num().

The completion code needs to determine the word the curser is
currently on.  Libreadline only provides the start and end position
of the current word in the line buffer, but not the word number.

This patch adds compute_word_num() to string.c which uses the same
algorithm as create_argv() to determine the word boundaries.

8 years agoconfigure: Introduce checks for libreadline.
Andre Noll [Thu, 25 Aug 2011 19:53:42 +0000 (21:53 +0200)]
configure: Introduce checks for libreadline.

This adds interactive.o to the object list of para_client if readline
was found and links para_client against libreadline in this case.

The new source files interactive.c and interactive.h are empty for now.

8 years agoclient: Split client_open() and client_close().
Andre Noll [Sun, 4 Sep 2011 16:41:40 +0000 (18:41 +0200)]
client: Split client_open() and client_close().

For interactive para_client sessions, we set up the client
configuration from the values given at the command line and in the
config file just as we do for non-interactive mode. However, in
interactive mode we must be able to execute arbitrary many client
commands using the same configuration. client_open() sets up the
configuration from the command line arguments and client_close()
destroys the configuration, so both functions do more than what is
needed in interactive mode.

This patch splits client_open() into two functions
client_parse_config() and client_connect(). The old client_open()
remains with the same semantics as before, it just calls both of the
new functions().

In the same spirit, client_close() is split into client_disconnect(),
which closes the file descriptor and deallocates the ressources of
the current command but does not free the configuration.

This allows to re-use the client configuration multiple times for
interactive sessions.