10 years agosched: Replace sched_shutdown() by task_notify_all().
Andre Noll [Sun, 8 Jul 2012 18:57:24 +0000 (18:57 +0000)]
sched: Replace sched_shutdown() by task_notify_all().

sched_shutdown() was mis-designed from the beginning as the registered
tasks have no chance to clean up. Using task notifiers allows to
replace the single caller of sched_shutdown().

10 years agosched: Introduce task notifications.
Andre Noll [Sun, 8 Jul 2012 18:57:16 +0000 (18:57 +0000)]
sched: Introduce task notifications.

Currently there is no canonical way to perform inter-task
communications. This leads to ugly hacks like kill_all_decoders()
of audiod which is called when audiod switches to "off" or "standby"
mode. This function sets the error state of the receiver task to a
negative value and removes the receiver's buffer tree node, magically
knowing that this is the correct action to take in order to shut down
the receiver cleanly.

This patch provides a simple infrastructure for task communications
which makes such hacks unnecessary. Tasks may send a notification
value, which is a standard error code, to another task by calling the
new task_notify() function. The receiving task is supposed to check
for any pending notifications in its ->post_select function.

As a first application of the notification concept, the above mentioned
kill_all_decoders() function is changed to use notifications instead.

10 years agoi9e_attach_to_stdout(): Don't insist on btrn == NULL.
Andre Noll [Thu, 12 Apr 2012 04:21:56 +0000 (06:21 +0200)]
i9e_attach_to_stdout(): Don't insist on btrn == NULL.

This allows to call this function even if the stdout buffer tree node
is already attached. In this case i9e_attach_to_stdout() detaches
the current btr node first.

10 years agoInteractive: Assorted whitespace/newline fixes.
Andre Noll [Sun, 8 Apr 2012 03:03:10 +0000 (05:03 +0200)]
Interactive: Assorted whitespace/newline fixes.

This avoids printing unnecessary newlines and fixes the
"double-prompt" bug.

10 years agointeractive: kill i9ep->line_handler_running.
Andre Noll [Mon, 9 Apr 2012 04:10:50 +0000 (06:10 +0200)]
interactive: kill i9ep->line_handler_running.

What we are interested in is not if the line handler is currently
running but if there exists a buffer tree node for the stdout task. So
check this condition instead and remove the unnecessary variable.

10 years agointeractive: Do not close stderr in i9e_close().
Andre Noll [Mon, 9 Apr 2012 04:12:14 +0000 (06:12 +0200)]
interactive: Do not close stderr in i9e_close().

Closing stderr prevents further log messages from being printed.

10 years agoAdd missing documentation of send_strerror().
Andre Noll [Sun, 4 Nov 2012 12:38:59 +0000 (13:38 +0100)]
Add missing documentation of send_strerror().

10 years agoMerge branch 't/filter_parse_config_cleanup'
Andre Noll [Sun, 18 Nov 2012 19:22:30 +0000 (20:22 +0100)]
Merge branch 't/filter_parse_config_cleanup'

8e290f Cleanup filter command line parsers.

10 years agoMerge branch 't/fade_improvements'
Andre Noll [Sun, 4 Nov 2012 13:20:36 +0000 (14:20 +0100)]
Merge branch 't/fade_improvements'

Was cooking for almost a month, let's merge it.

483d31 fade: Upgrade client command log message to loglevel NOTICE.
49412f fade: Handle non-positive fade time gracefully.
376456 fade: Add --loglevel option.
dec704 fade: Generate fade.ggo from m4 template.
0968b2 fade: Introduce --mixer-api to choose between ALSA and OSS.
b4171b fade: Add the ALSA mixer implementation.
46af7d fade: Switch to modular mixer API.
702457 fade: Infrastructure for the modular mixer API.
3a2580 fade: Abort on client command failures.
b618da fade: Remove some unnecessary includes.
f502e6 configure: Add fade.o to object list only if oss was detected.

10 years agoCleanup filter command line parsers.
Andre Noll [Wed, 10 Oct 2012 07:29:04 +0000 (09:29 +0200)]
Cleanup filter command line parsers.

All these functions call the gengetopt parser which aborts on errors.
Hence it is pointless to check the return value of the parser.

The patch also renames most of the *args_info structs to "conf".

10 years agoamp: Remove pointless private_amp_data->conf.
Andre Noll [Tue, 9 Oct 2012 21:07:24 +0000 (23:07 +0200)]
amp: Remove pointless private_amp_data->conf.

There is no need to store the configuration of the amp filter
in its privata_data structure since we can always get it via
the filter node pointer.

10 years agobuffer_tree.c: Fix a typo.
Andre Noll [Sun, 21 Oct 2012 09:31:57 +0000 (11:31 +0200)]
buffer_tree.c: Fix a typo.

10 years agobtr_next_buffer_omit(): Add comment on passing NULL pointer.
Andre Noll [Tue, 9 Oct 2012 21:05:44 +0000 (23:05 +0200)]
btr_next_buffer_omit(): Add comment on passing NULL pointer.

It has always been OK to pass a NULL buffer pointer to this function,
but this was not documented yet.

10 years libflac also affects para_afh and para_server.
Andre Noll [Mon, 8 Oct 2012 20:54:44 +0000 (22:54 +0200)] libflac also affects para_afh and para_server.

If libflac was not detected, there will be no flac support
whatsoever. The current warning message is misleading since
it only mentions para_audiod and para_filter.

10 years agofade: Upgrade client command log message to loglevel NOTICE.
Andre Noll [Mon, 16 Jul 2012 19:53:35 +0000 (21:53 +0200)]
fade: Upgrade client command log message to loglevel NOTICE.

This is one of the more interesting log messages of para_fade.

10 years agofade: Handle non-positive fade time gracefully.
Andre Noll [Sun, 15 Jul 2012 21:57:53 +0000 (23:57 +0200)]
fade: Handle non-positive fade time gracefully.

Currently, we don't do anything if the fade_time parameter is less
than or equal to zero. Just setting the destination volume seems to
be more natural in this case.

10 years agofade: Add --loglevel option.
Andre Noll [Sun, 15 Jul 2012 21:37:46 +0000 (23:37 +0200)]
fade: Add --loglevel option.

All the infrastructure is already in place, so this patch is
quite simple.

10 years agofade: Generate fade.ggo from m4 template.
Andre Noll [Sun, 15 Jul 2012 21:30:00 +0000 (23:30 +0200)]
fade: Generate fade.ggo from m4 template.

This allows to use the generic --config file template.

10 years agofade: Introduce --mixer-api to choose between ALSA and OSS.
Andre Noll [Sun, 15 Jul 2012 17:58:59 +0000 (19:58 +0200)]
fade: Introduce --mixer-api to choose between ALSA and OSS.

Currently, if both OSS and ALSA are supported, there is no way to
select the OSS mixer API.

This adds a new option to para_fade which allows to force the
given mixer API.

10 years agofade: Add the ALSA mixer implementation.
Andre Noll [Mon, 16 Jul 2012 19:25:51 +0000 (21:25 +0200)]
fade: Add the ALSA mixer implementation.

This provides another mixer API which relies on the libasound library
of ALSA.

The manual is changed accordingly to mention that para_fade now
supports both OSS and ALSA.

10 years agofade: Switch to modular mixer API.
Andre Noll [Mon, 16 Jul 2012 19:29:19 +0000 (21:29 +0200)]
fade: Switch to modular mixer API.

Although only the oss mixer is supported ATM, this commit paves the way
for supporting more than one mixer API. It moves the oss-specific part
of para_fade to the new oss_mix.c file added by the previous commit.

Different mixer APIs will support different sets of mixer channels,
so handling of the various mixer channels is changed from a ggo enum
config option to a string option which allows to specify arbitrary
strings. The mixer implementations must check the given strings
themselves and are supposed to define a ->get_channels method which
returns the available channels. This function is called from fade.c
if an invalid channel is given.

The default mixer device, currently hardcoded in the ggo file to
"/dev/mixer", is fine for OSS. The ALSA mixer will need another default
however. Therefore this patch removes the default setting from the ggo
file so that the corresponding ggo config variable will be NULL if no
mixer device was given. The mixer implementations are supposed to check
this variable and fall back to their individual defaults if it is NULL.

10 years agofade: Infrastructure for the modular mixer API.
Andre Noll [Sun, 15 Jul 2012 11:08:28 +0000 (13:08 +0200)]
fade: Infrastructure for the modular mixer API.

This commit prepares for supporting more than one mixer by introducing
$mixers in which contains the set of supported mixers. It
creates an empty oss_mix.c file to which the oss-specific code will
be moved in the next patch.

The fade-specific stuff of is moved to a separate section
since whether to build fade will soon no longer depend on OSS only.

The patch is a bit large due to some generic m4 macros being moved to
the top of as they are needed in the new fade section.

10 years agofade: Abort on client command failures.
Andre Noll [Mon, 2 Jul 2012 06:40:31 +0000 (08:40 +0200)]
fade: Abort on client command failures.

Currently, we check only the exit code of wait(), but do not look
at the exit status of the child. So the fade command continues
even if all para_client commands fail.

Fix this by investigating the termination status of the child

10 years agofade: Remove some unnecessary includes.
Andre Noll [Sun, 15 Jul 2012 12:31:02 +0000 (14:31 +0200)]
fade: Remove some unnecessary includes.

These are included already from para.h.

10 years agoconfigure: Add fade.o to object list only if oss was detected.
Andre Noll [Sun, 15 Jul 2012 09:24:35 +0000 (11:24 +0200)]
configure: Add fade.o to object list only if oss was detected.

The all_errlist_obj variable should only contain objects which can
actually be compiled on the given setup.

However, fade.o was added to this variable unconditionally. This patch
makes it conditional on oss.

10 years agoNEWS update.
Andre Noll [Sun, 7 Oct 2012 09:11:02 +0000 (11:11 +0200)]
NEWS update.

11 years agoSimplify ggo makefile.
Andre Noll [Sun, 29 Jul 2012 18:51:18 +0000 (20:51 +0200)]
Simplify ggo makefile.

Currently we have three different targets for creating *_cmdline.[ch]
files. This is because receivers, filters and writers need slightly
different command line options.

This patch defines the common options in the ggo makefile and moves
additional parameters to the individual .m4 files so that a single
target to create *_cmdline.[ch] is now sufficient.

The name of the command line parsers of some filters and writers
changed due to this unification, so these are updated accordingly.

11 years agowrite: Get rid of gengetopt's string parser.
Andre Noll [Sun, 29 Jul 2012 17:49:46 +0000 (19:49 +0200)]
write: Get rid of gengetopt's string parser.

It causes gengetopt to generate quite some additional code for
the string parsers of all writers. Moreover, this string parser is
inferior to create_argv() and create_shifted_argv() of string.c as
gengetopt's parser does not honor any quoting at all.

This commit changes the signature of the ->parse_config_or_die method
of struct writer to take an (argc, argv) pair instead of a string.  All
writers can thus call the vanilla command line parser of gengetopt. The
single user in write_common.c now calls ->parse_config_or_die()
with an (argc, argv) pair obtained from create_shifted_argv().

11 years agoIntroduce create_shifted_argv().
Andre Noll [Sun, 29 Jul 2012 16:39:58 +0000 (18:39 +0200)]
Introduce create_shifted_argv().

In order to create argument vectors for the gengetopt parsers,
filter_common.c and recv_common.c call create_argv() on the given
--filter or --receiver option, then shift the returned array by one
to the right because gengetopt expects the program name in argv[0].

This commit creates a new helper function for this purpose and changes
the two callers to use the new function.

11 years agoFix off-by-one bug in create_argv().
Andre Noll [Sun, 29 Jul 2012 15:55:37 +0000 (17:55 +0200)]
Fix off-by-one bug in create_argv().

Nothing serious, just wasting memory.

11 years agoggo: Move options from makefile to .m4 files.
Andre Noll [Sun, 29 Jul 2012 15:13:10 +0000 (17:13 +0200)]
ggo: Move options from makefile to .m4 files.

It's easier to specify these options for gengetopt in the
corresponding m4 files, together with the command line
options of the program.

11 years agopara_gui ggo: Do not pass --no-handle-error.
Andre Noll [Sun, 29 Jul 2012 14:28:28 +0000 (16:28 +0200)]
para_gui ggo: Do not pass --no-handle-error.

We are exiting anyway on errors. This change allows to skip
the check of the return value of gui_cmdline_parser() and
removes a line from the makefile.

11 years agoRemove fsck.m4.
Andre Noll [Sun, 29 Jul 2012 14:18:21 +0000 (16:18 +0200)]
Remove fsck.m4.

This should have removed in 0.4.0 back when osl was moved to a
separate library.

11 years agoPut dependency files to a separate directory.
Andre Noll [Sat, 28 Jul 2012 16:14:24 +0000 (18:14 +0200)]
Put dependency files to a separate directory.

Currently the dependency files are stored in the same directory as the
object files. This moves these files to the new build/deps directory
instead, and makes "make clean" remove the objects directory while
"make clean2" now removes the whole build directory.

As a side effect of this change, the $(all_objs) variable now contains
only the non-directory part of the objects, so we may get rid of the
directory-stripping in the test makefile.

11 years agoMakefile: Simplify option parsing for verbose build.
Andre Noll [Sat, 28 Jul 2012 09:03:28 +0000 (11:03 +0200)]
Makefile: Simplify option parsing for verbose build.

There is no point in introducing the intermediate BUILD_VERBOSE
variable. Just set $Q directly and make it a singly expanded variable.

11 years agoconfigure: Replace AC_TRY_RUN and AC_TRY_LINK.
Andre Noll [Sat, 28 Jul 2012 06:44:00 +0000 (08:44 +0200)]
configure: Replace AC_TRY_RUN and AC_TRY_LINK.

Both macros are considered obsolete. AC_TRY_RUN has been obsoleted
by AC_RUN_IFELSE while AC_TRY_LINK has been replaced by AC_LINK_IFELSE.

This patch replaces all occurrences of these two obsolete macros and
bumps the required autoconf version to 2.61 (which was released in
2006 and is shipped with Ubuntu Hardy).

11 years agoconfigure: Replace AC_HELP_STRING by AS_HELP_STRING.
Andre Noll [Sun, 22 Jul 2012 20:37:58 +0000 (22:37 +0200)]
configure: Replace AC_HELP_STRING by AS_HELP_STRING.

The former is considered obsolete. Reported by

autoconf --warnings obsolete

11 years agoconfigure: Remove check for return value of signal handlers.
Andre Noll [Sun, 22 Jul 2012 20:29:49 +0000 (22:29 +0200)]
configure: Remove check for return value of signal handlers.

These days one may safely assume C89 semantics where the type of a
signal handler is void.

11 years Remove call to aclocal.
Andre Noll [Sun, 22 Jul 2012 19:54:07 +0000 (21:54 +0200)] Remove call to aclocal.

aclocal is part of automake, which is not used at all by the paraslash
build system.

11 years agoAllow to specify build dir via make O=<dir>.
Andre Noll [Sun, 22 Jul 2012 18:09:37 +0000 (20:09 +0200)]
Allow to specify build dir via make O=<dir>.

11 years agoMakefile: Move generated files to build directory.
Andre Noll [Thu, 26 Jul 2012 02:28:37 +0000 (04:28 +0200)]
Makefile: Move generated files to build directory.

This instructs the build system to store the generated ggo files,
all objects and depend files, and the man pages in the new "build"

11 years agoMakefile: Don't hardcode path for generated man pages.
Andre Noll [Thu, 26 Jul 2012 02:28:56 +0000 (04:28 +0200)]
Makefile: Don't hardcode path for generated man pages.

Using $(man_dir) throughout makes it easier to change the name of
this directory.

11 years agoMakefile: Clean up directory creation.
Andre Noll [Sun, 22 Jul 2012 14:33:20 +0000 (16:33 +0200)]
Makefile: Clean up directory creation.

This combines all directory targets into a single target and executes
the contents of the new MKDIR_P variable (set to "mkdir -p") to create
the directory.

11 years agoMove gengetopt input files to m4 directory.
Andre Noll [Sun, 22 Jul 2012 14:11:11 +0000 (16:11 +0200)]
Move gengetopt input files to m4 directory.

This separates the gengetopt input directory from its output directory.
All gengetopt m4 file are moved to the new m4/gengetopt directory.

As a consequence of this cleanup, we may get rid of the .gitignore
file for the generated ggo files, which is a good thing as it was
hard to keep it up to date.

11 years agoDifferentiate between ggo_dir and m4_ggo dir.
Andre Noll [Sun, 22 Jul 2012 13:39:13 +0000 (15:39 +0200)]
Differentiate between ggo_dir and m4_ggo dir.

The former is the output directory for the generated ggo files
while the latter is the input directory containing the .m4 files.

Separating the two allows to switch to different directories for
input and output files.

11 years agoTreat all ggo files as m4 files.
Andre Noll [Sun, 22 Jul 2012 13:15:52 +0000 (15:15 +0200)]
Treat all ggo files as m4 files.

Most input files for gengetopt are already generated from corresponding
m4 files. This renames the remaining ones, so that all .ggo files
are now generated and can be removed by the make clean variants.
Hence we do not need to track the set of generated ggo files any more.

11 years agoMerge branch 't/releases_branch'
Andre Noll [Sun, 26 Aug 2012 12:26:00 +0000 (14:26 +0200)]
Merge branch 't/releases_branch'

18d2c3 Remove all release tarballs from the master branch.
f82ac2 web: Rename tarball download directory.

Was cooking since 2012-07-28, tested also on athcx.

11 years agosignal: Improve documentation of para_signal_init().
Andre Noll [Tue, 14 Aug 2012 11:09:43 +0000 (13:09 +0200)]
signal: Improve documentation of para_signal_init().

para_signal_init() does not install any signal handlers, so this part of the
documentation is incorrect. The patch fixes this flaw, mentions which end of
the signal pipe is returned to the caller and improves the wording a bit.

11 years agoMerge remote-tracking branch 'boock/master'
Andre Noll [Mon, 13 Aug 2012 08:08:41 +0000 (10:08 +0200)]
Merge remote-tracking branch 'boock/master'

11 years agoMerge branch 'maint'
Andre Noll [Mon, 13 Aug 2012 08:08:28 +0000 (10:08 +0200)]
Merge branch 'maint'

11 years agoexec.c: Remove close_on_fork include.
Andre Noll [Mon, 13 Aug 2012 06:31:13 +0000 (08:31 +0200)]
exec.c: Remove close_on_fork include.

It does not depend on it.

11 years agoMerge branch 'maint'
Andre Noll [Mon, 13 Aug 2012 08:04:40 +0000 (10:04 +0200)]
Merge branch 'maint'

11 years agosideband: Fix SBB_INIT().
Andre Noll [Mon, 13 Aug 2012 06:35:17 +0000 (08:35 +0200)]
sideband: Fix SBB_INIT().

The macro parameter is called "_band" rather than "band". This
only worked by accident because all users called the argument

11 years agoserver.ggo depends on config_file.m4.
Andre Noll [Tue, 7 Aug 2012 11:27:11 +0000 (13:27 +0200)]
server.ggo depends on config_file.m4.

server.m4 includes config_file.m4 but this dependency was not listed
in the ggo makefile.

11 years agoserver.ggo depends on config_file.m4.
Andre Noll [Tue, 7 Aug 2012 11:27:11 +0000 (13:27 +0200)]
server.ggo depends on config_file.m4.

server.m4 includes config_file.m4 but this dependency was not listed
in the ggo makefile.

11 years agoRemove all release tarballs from the master branch.
Andre Noll [Sun, 22 Jul 2012 10:13:32 +0000 (12:13 +0200)]
Remove all release tarballs from the master branch.

It made downloads of the development snapshots via gitweb very large.
All release tarballs are still available in the separate "releases"

11 years agoweb: Rename tarball download directory.
Andre Noll [Sun, 22 Jul 2012 10:14:20 +0000 (12:14 +0200)]
web: Rename tarball download directory.

"releases" seems to be a better name than "versions". The web sync
script (in another repo) has been changed accordingly.

11 years agoMakefile: Move up the definition of various directories.
Andre Noll [Sun, 22 Jul 2012 12:49:18 +0000 (14:49 +0200)]
Makefile: Move up the definition of various directories.

This way we have all of them at one place in the Makefile.

11 years agoRemove some stale entries from .gitignore.
Andre Noll [Sun, 22 Jul 2012 15:53:24 +0000 (17:53 +0200)]
Remove some stale entries from .gitignore.

11 years Remove unnecessary mkdir command.
Andre Noll [Sun, 22 Jul 2012 15:46:06 +0000 (17:46 +0200)] Remove unnecessary mkdir command.

This crept in five years ago (commit 4c034f8d) for no good reason.

11 years agoMakefile: Remove targets for html man pages.
Andre Noll [Sun, 22 Jul 2012 14:21:28 +0000 (16:21 +0200)]
Makefile: Remove targets for html man pages.

The html version of the man pages is generated by other means these
days, so the two targets can be removed.

11 years agoMakefile: Remove $all_ggos.
Andre Noll [Sun, 22 Jul 2012 12:40:09 +0000 (14:40 +0200)]
Makefile: Remove $all_ggos.

This variable is unused.

11 years agoserver: Remove "foo" feature.
Andre Noll [Mon, 23 Jul 2012 19:50:40 +0000 (21:50 +0200)]
server: Remove "foo" feature.

Oops, this feature was added only for testing feature negotiation
with multiple features and should have been deleted before 0.4.11 was
released. This was even mentioned in the commit message of e151dbb7,
which introduced the bogus "foo" feature..

11 years agocom_init(): Send error message only for non-sideband connections.
Andre Noll [Tue, 24 Jul 2012 06:42:23 +0000 (08:42 +0200)]
com_init(): Send error message only for non-sideband connections.

If com_init() fails, this non-sideband error message is currently
sent also for sideband connections, which confuses the client. Send
it only for non-sideband connections.

11 years agofade: Fix command for switching moods.
Andre Noll [Mon, 23 Jul 2012 06:41:23 +0000 (08:41 +0200)]
fade: Fix command for switching moods.

It contained a newline, which was not a problem before the sideband
API was introduced. However, with sideband being the default, the
newline character is interpreted as part of the name of the mood or
playlist, causing the mood switch to fail.

The fix is simple: Just remove the trailing newline character.

11 years agoFix typo in t0003.
Andre Noll [Sun, 22 Jul 2012 10:43:32 +0000 (12:43 +0200)]
Fix typo in t0003.

11 years agoNEWS: Whitespace and typo fixes.
Andre Noll [Fri, 20 Jul 2012 20:09:27 +0000 (22:09 +0200)]
NEWS: Whitespace and typo fixes.

paraslash-0.1.0 was released in 2004 rather than 2204.

11 years agonew codename, reset version to git
Andre Noll [Fri, 20 Jul 2012 08:46:02 +0000 (10:46 +0200)]
new codename, reset version to git

11 years agothe paraslash-0.4.11 release tarball
Andre Noll [Fri, 20 Jul 2012 08:41:24 +0000 (10:41 +0200)]
the paraslash-0.4.11 release tarball

11 years agoparaslash 0.4.11 v0.4.11
Andre Noll [Fri, 20 Jul 2012 08:38:07 +0000 (10:38 +0200)]
paraslash 0.4.11

11 years agoMerge branch 't/btr_improvements'
Andre Noll [Thu, 19 Jul 2012 16:46:58 +0000 (18:46 +0200)]
Merge branch 't/btr_improvements'

146316 btr_exec_up(): Also ask given node.
6d9c35 btr: Introduce btr_add_output_dont_free().
4ac313 btr: Remove btr_free_node().

Has been cooking in next for a week.

11 years agofade: Use proper ldflags.
Andre Noll [Sun, 15 Jul 2012 09:35:04 +0000 (11:35 +0200)]
fade: Use proper ldflags.

This cut and paste typo caused para_fade to be linked against
the libraries of para_write on systems with libossaudio (BSD).

11 years agofade: Fix sleep time in snooze mode.
Andre Noll [Mon, 2 Jul 2012 06:38:00 +0000 (08:38 +0200)]
fade: Fix sleep time in snooze mode.

We slept for the wrong amount of seconds.

11 years agoMerge branch 't/wma_improvements'
Andre Noll [Wed, 18 Jul 2012 16:47:33 +0000 (18:47 +0200)]
Merge branch 't/wma_improvements'

a02611 wma: Remove exponent_sizes[] from struct private_wma_data.

Has been cooking long enough.

11 years agogui: Add missing dependency on included m4 file to ggo makefile.
Andre Noll [Sun, 15 Jul 2012 21:22:33 +0000 (23:22 +0200)]
gui: Add missing dependency on included m4 file to ggo makefile.

gui.ggo depends on config_file.m4, but this dependency was not listed
in the makefile.

11 years agoMerge branch 't/old_gcc_fixes'
Andre Noll [Sun, 15 Jul 2012 19:57:33 +0000 (21:57 +0200)]
Merge branch 't/old_gcc_fixes'

82d304 buffer_tree.h: Fix gcc warning.
5535f6 alsa: Fix gcc warning.

Simple enough and well tested with no regressions found.

11 years agoMerge branch 't/clang'
Andre Noll [Wed, 11 Jul 2012 23:53:14 +0000 (01:53 +0200)]
Merge branch 't/clang'

869f0e aft.c: Silence scan-build warning.
4cdf53 mood: Improve readability of get_item_score().
ec5040 mood: Add score value only for admissible files.
76d527 client: Remove pointless assignment.
466c9b gcrypt: Remove pointless assignment.
98c13e gcrypt: Remove dead stores.
d458c2 mp3dec: Remove dead store.

Cooking for quite some time - no problems.

11 years agobtr_exec_up(): Also ask given node.
Andre Noll [Sun, 8 Apr 2012 19:31:10 +0000 (21:31 +0200)]
btr_exec_up(): Also ask given node.

At the moment, the buffer tree walk starts at the parent of the given
node. Users of the btr API can not execute a command for a node unless
it has at least one child.

This unnecessary restriction is removed in this commit by letting
the tree walk start at the given node rather than its parent.
This actually simplifies the code a bit.

11 years agobtr: Introduce btr_add_output_dont_free().
Andre Noll [Tue, 23 Aug 2011 19:04:02 +0000 (21:04 +0200)]
btr: Introduce btr_add_output_dont_free().

At the moment, all buffers which are fed to a buffer tree must be
allocated on the heap since the buffer tree code automatically frees
the buffer once its refcount dropped to zero.

The new afh receiver, however, mmaps the audio file and likes to feed
chunks of this memory map into the buffer tree. This is currently
impossible because such buffers must not be freed.

This patch adds the new public function btr_add_output_dont_free()
which works like btr_add_output() but sets the new dont_free bit
which prevents the buffer from being deallocated.

Also btr_inplace_ok() is changed to return "false" whenever there
exists a buffer in the input queue with the dont_free bit set.

11 years agowrite: Remove ->shutdown.
Andre Noll [Mon, 25 Jun 2012 11:16:51 +0000 (13:16 +0200)]
write: Remove ->shutdown.

No writer implements this method, and nobody even looks at this
pointer. Remove it.

11 years agobtr: Remove btr_free_node().
Andre Noll [Mon, 9 Apr 2012 19:57:54 +0000 (21:57 +0200)]
btr: Remove btr_free_node().

This has turned out to be source for bugs. Deallocate everything
in btr_remove_node() hence making removing the node and freeing
its resources an atomic operation.

To avoid dangling pointers to freed btrn nodes, the argument of
btr_remove_node() is changed to to struct btr_node **btrnp.

11 years agoaudiod: Kill some dead code.
Andre Noll [Sun, 1 Jul 2012 18:55:44 +0000 (20:55 +0200)]
audiod: Kill some dead code.

Both conditions are never true at this point.

11 years agoMerge branch 't/command_util_cleanups'
Andre Noll [Sun, 8 Jul 2012 16:02:54 +0000 (16:02 +0000)]
Merge branch 't/command_util_cleanups'

9f58f7 command_util: grep source files only once.
b4aff8 command_util: Fix make dependencies for *_command_list.h.
289547 Make (most) command handlers static.
ad3864 command_util: Remove code for generating .c files.
c9df5d command_util: Write array definition in header mode.
73ca79 command_util: Output array members with backslashes.
3792a2 command_util: Avoid empty output lines.
a51586 command_util: replace dump_xxx() by make_xxx().
6a4f5e command_util: Kill HC and CC header lines.

11 years agoMerge branch 't/alsa_fix'
Andre Noll [Mon, 2 Jul 2012 06:52:16 +0000 (08:52 +0200)]
Merge branch 't/alsa_fix'

3a56a6 alsa: Do not buffer more than 500ms.

Seems to work fine everywhere.

11 years agocommand_util: grep source files only once.
Andre Noll [Sun, 15 Apr 2012 05:25:22 +0000 (07:25 +0200)]
command_util: grep source files only once.

Currently we grep once for each template member. This patch speeds
up the header mode of by a factor of 3.

11 years agocommand_util: Fix make dependencies for *_command_list.h.
Andre Noll [Mon, 9 Apr 2012 23:43:38 +0000 (01:43 +0200)]
command_util: Fix make dependencies for *_command_list.h.

The *_command_list.h files depend on the .c files listed in the SF
header line of the .cmd command file because gets
the declaration of the command handlers from these files.

Unfortunately, make does not know about these dependencies, so we
have to mention them in the Makefile as well.

11 years agoMake (most) command handlers static.
Andre Noll [Sat, 7 Apr 2012 06:30:27 +0000 (08:30 +0200)]
Make (most) command handlers static.

Now that the array of commands for para_server and para_audiod are
read from a header file, the command handlers can be made static if
they are in the same file that defines the command array.

This patch makes all such handlers static and tweaks
to recognize also static functions.

11 years agocommand_util: Remove code for generating .c files.
Andre Noll [Thu, 28 Jun 2012 02:45:09 +0000 (04:45 +0200)]
command_util: Remove code for generating .c files.

Now that the *_command_list.c files are no longer needed we may remove
this mode from This makes quite a few headers of
the .cmd files unused, so these can be removed as well.

11 years agocommand_util: Write array definition in header mode.
Andre Noll [Thu, 28 Jun 2012 02:23:32 +0000 (04:23 +0200)]
command_util: Write array definition in header mode.

This modifies so that when it is called in header mode,
it also defines a macro for the array of all commands. command.c and
audiod_command.c are changed to use the new preprocessor macro.

This allows to get rid of the *command_list.c files completely. We may
hence remove these files from the various lists of errlist objects
in and from error.h. A subsequent patch will clean up
the parts that have become unused by this commit.

11 years agocommand_util: Output array members with backslashes.
Andre Noll [Thu, 28 Jun 2012 01:08:00 +0000 (03:08 +0200)]
command_util: Output array members with backslashes.

This makes sure that the generated code for the definition of the
command array has backslashes at the end of each line.  Currently
this does not matter because make_array_member() is only called when
generating C code where new lines don't matter.

However, ending each line with a backslash allows to use the same
output for a cpp #define which can go into the header file.

11 years agocommand_util: Avoid empty output lines.
Andre Noll [Thu, 28 Jun 2012 00:51:28 +0000 (02:51 +0200)]
command_util: Avoid empty output lines.

Additional empty output lines do not matter for the generated C code or
the function declarations in the header files.  However, a subsequent
commit changes to  generate a preprocessor macro for
the header file, and there we can not afford empty lines unless they
end with a backslash to continue the line.

This patch makes sure that no superfluous newlines are printed in
the output.

11 years agocommand_util: replace dump_xxx() by make_xxx().
Andre Noll [Wed, 27 Jun 2012 23:18:23 +0000 (01:18 +0200)]
command_util: replace dump_xxx() by make_xxx().

This replace dump_array_member(), dump_completion() and dump_proto()
by their make_xxx counterpart.

This allows to iterate over all commands twice.

11 years agocommand_util: Kill HC and CC header lines.
Andre Noll [Wed, 27 Jun 2012 22:57:52 +0000 (00:57 +0200)]
command_util: Kill HC and CC header lines.

There is no point in pretty-printing the generated files. Nobody reads
these files anyway and doxygen ignores them as well.

11 years agowma: Remove exponent_sizes[] from struct private_wma_data.
Andre Noll [Fri, 1 Jun 2012 08:56:10 +0000 (10:56 +0200)]
wma: Remove exponent_sizes[] from struct private_wma_data.

This array is only used in compute_scale_factor_band_sizes(), and
even there we need only the current value. So it is enough to have
a single local variable in this function.

11 years agosideband: Fix use after free bug.
Andre Noll [Fri, 29 Jun 2012 14:28:11 +0000 (16:28 +0200)]
sideband: Fix use after free bug.

Commit 32ffc06c (Pass command exit status via sideband to client.)
made para_server's command handler send the exit status of the command
to the client as an RC4-encrypted sideband packet.

However, the encryption key has already been freed when this packet
is sent. This may result in client errors about invalid or unexpected
sideband designators, but these errors occurred rather infrequently,
which makes this bug hard to trigger. Presumably the freed memory is
usually not being reused since the command handler is going to dye
anyway after the packet is sent.

This patch fixes the bug by deferring to free the crypto keys until
the exit code has been sent.

11 years agoMerge branch 't/decoder_fixes'
Andre Noll [Tue, 26 Jun 2012 20:07:55 +0000 (22:07 +0200)]
Merge branch 't/decoder_fixes'

97dbfe wmadec: Only decode one superframe at a time.
d6e017 flacdec: Only process a single flac audio frame.
1e1968 flacdec_close(): Be liberal in what you accept.
3f96f9 oggdec: Fix EOF handling on repositioning.
c7e2f7 oggdec: Realloc buffer to save memory.
634e75 oggdec: Do not decode more than necessary.

These changes are well tested and there are no known problems.

11 years agobuffer_tree.h: Fix gcc warning.
Andre Noll [Mon, 18 Jun 2012 17:44:03 +0000 (19:44 +0200)]
buffer_tree.h: Fix gcc warning.

"loglevel" is declared as a global variable elsewhere, and gcc-3.3.3 complains
beacuse of this:

buffer_tree.h:199: warning: declaration of `loglevel' shadows a global declaration

Just call the thing "ll".

11 years agoalsa: Fix gcc warning.
Andre Noll [Mon, 18 Jun 2012 17:43:34 +0000 (19:43 +0200)]
alsa: Fix gcc warning.

gcc-3.3.3 complains about the local variable "log" in alsa_write.c:

alsa_write.c:75: warning: declaration of `log' shadows a global declaration

Rename this variable to "output_log" to silence the warning.

11 years agoMerge branch 't/sb'
Andre Noll [Sun, 24 Jun 2012 18:53:23 +0000 (20:53 +0200)]
Merge branch 't/sb'

Was cooking in next for several weeks with no problems so far.

fd864c Fix documentation of pass_buffer_as_shm().
32ffc0 Pass command exit status via sideband to client.
7dc1b5 Send command output via sideband.
0229d7 Introduce send_strerror().
d5a9e8 Simplify pass_buffer_as_shm().
3667d7 command.c: Improve get_status().
aa3fe7 Pass command via sideband.
9c2a26 Use sideband also for "proceed" handshake.
4744d9 Use sideband also for challenge response.
fa9e0e Send and receive challenge via sideband.
2830b9 Add sideband implementation.
e151db Implement client-server feature negotiation.
e31d5d afs: Rename sc_send_result() to afs_cb_result_handler().
68cb0a Introduce afs_max_size_handler_data and afs_max_size_handler().
d04b83 client: Improve error diagnostics.

11 years agoFix documentation of pass_buffer_as_shm().
Andre Noll [Mon, 18 Jun 2012 06:15:18 +0000 (08:15 +0200)]
Fix documentation of pass_buffer_as_shm().

Commit d5a9e8 renamed the argument "fd_ptr" to "fd" but missed
to rename this variable also in the long description.