rename plm_dbtool.c to playlist_selector.c

Rename "dbtool" status item to "selector"

Change doxygen file description from dbtool to selector

Another trivial rename

gui,sdl_gui: change the prefix from "dbtool" to "selector"

trivial dbtool->selector renames

Rename some local variables in command.c

rename struct dbtool to audio_file_selector

db.h: Kill an unused enum

Rename com_cdt to com_chs

chs: change (audio file) selector.

dbtool -> audio file selector renaming

This updates only Documentation and log messages.

Rename the --dbtool option to --selector

As two out of the three "database tools" of paraslash don't use a
database at all, let's call them "audio file selectors" instead.

ogg.c: don't use ogg_int64_t

ssize_t should do equally well and avoids gcc warnings.

whitespace fixes

audiod: fix a memory leak in ringbuffer handling

Found by valgrind.

web/header2.html: update the changes menu item to point to gitweb.

From within the doxygen API reference that menu item still pointed
to the old cvs changelog which is no longer present.

Merge branch 'no_colon_separators'

Convert demo-script to the new syntax without colons.

As this won't work with paraslash-0.2.10, change the script
to download the git tarball.

Add a missing header file to Makefile.in

ipc.h was not included in the tarball. With this fix, the nightly git
snapshot compiles again.

maintainer-clean: also remove skencil/*.ps skencil/*.pdf and doc

header2.html is needed by doxygen

kill a pointless "optind = 1".

Merge branch 'xf_64_compile_warning_fixes'

README: Mention the name of some debian packages

libasound2-dev is needed for para_play, ogg vorbis support requires
libogg-dev and libvorbis-dev. Thanks to Steffen Klassert for this info.

ogg.c: use %z modifier

oggdec.c: %d -> %zd fixes

another %d -> %zd fix

Noticed by Steffen Klassert

net.h: use proper type for the addrlen parameter of para_accept() also in prototype

An earlier patch changed that parameter from size_t to socklen_t
but missed to update also the corresponding prototype in net.h. Thanks to
Steffen Klassert for pointing this out.

Simplify for_each_line()

It is always called with the num parameter being zero. So nuke that
parameter and update the docs.

Kill an outdated comment..

..which stated that filter command line options should be separated by colons.

para_filter: Update --help

The help text still used the old syntax with colons as separators. Fix that.

para_filter: Fix and improve help text for --list_filters

The old help text still used colons as separators. Mark those filters
which have their own command line parser with a star in the filter
list. Some trivial whitespace fixes also.

use %zd instead of %d for size_t

As of version 2.1 of glibc there is the %z modifier, so use it.
This avoids many warnings of the type

warning: int format, different type arg

on x86_64.

para_accept(): Use socklen_t instead of size_t

Fixes the following warning on x86_64:

net.c:76: warning: int format, different type arg (arg 4)

(no log message)

kill another superfluous '#include "gcc-compat.h"'

Commit 4b4c68de8d7390f966e46d9402d5499d2e8ee227 missed one of these.

fix off-by-one error

Also, make "bufsize" const instead of static.

no need to include gcc-compat.h twice

It gets already included from para.h.

make header_len an integer

This fixes the following warning:

ortp_send.c:169: warning: pointer targets in passing argument 1 of 'af->get_header_info' differ in signedness

simplify open_audiod_pipe(), remove a trailing newline which caused
the top window to flicker if para_audiod is not running.

Merge branch 'no_colon_separators'

Merge branch 'para_play_fixes'

Use whitespace instead of colons to separate receiver/filter args

This is much nicer but the receiver/filter argument must now be quoted if any
options are present.

another typo in comment

ipc.c:44: Warning: expected whitespace after param command

configurable input buffer size and prebuffering

play.c: pcm_write() aborts on errors. No need to check the return value.

Fix --device bug

Duuuh, the --device option never worked because "plug:swmix" was harcoded
in play.c instead of using the value from the configuration. Even worse,
the default in play.ggo was _also_ set to "plug:swmix". Replace that by
"plughw:0,0".

This patch also simplifies and optimizes play_pcm().

Anyway, software mixing is still recommended. Here's an aprropriate
/etc/asound.conf:

---------------------------------------------------- /etc/asound.conf
pcm.swmix {
type dmix
# any unique number here
ipc_key 313
        ipc_perm 0666

slave {
pcm "hw:0,0"
# these settings may require tweaking for different sound
# cards; this is for the Powerbook's built-in snd-powermac
# probably not required at all for well-behaved cards...
period_time 0
period_size 1024
buffer_size 8192
# mentioning rate fixes wrong speed/pitch in native ALSA stuff
# rate 44100
}
}

pcm.dsp0 {
type plug
slave.pcm "swmix"
}

ctl.mixer0 {
type hw
card 0
}

pcm.!default {
type plug
slave.pcm "swmix"
}
---------------------------------------------------------------------

a Use the
: Use a large input
buffer and For one, the

fix another typo in comment

announce gitweb

add more ipc documentation

Also, add a GPL header and replace "shared memory area" by "shared memory segment".

typo

replace link to the changelog by link to gitweb

add documentation of struct plm_client_data

replace cvs checkout instructions by git clone command

for those of us who are interested in history..

Typo

replace link to cvs snapshot by git snapshot

cvs->git update

Avoid busy loop if someone nasty removes the semaphores currently in use.

Just die if semop failed 500 times.

change version.

The shiny new git tree was still called "cvs".

mention plm and update the mysql description.

Also, selector is now the prefered word, rather than dbtool.

use the random dbtool as default

That simplifies the code quite a bit.

make para_server use the new ipc subsystem

Straight-forward conversion.

This BTW gets rid of the ancient /dev/zero hack which worked pretty
well for all the years but is not general enough for the plm dbtool.

Another 42 also disappeared ;)

fix the plm database tool

(Re)loading the playlist uses the new mutex and
shm helpers and is hopefully a race-free implementation.

It works as follows:

The plm init function (parent, server context) reserves
a shared memory area (client data) as well as two unlocked
mutexes. The first mutex serializes access for racing clients
that want to replace the playlist. The second mutex serializes
the access between parent and child (com_lpl()).

com_lpl() loads the playlist from the client into a tmp buffer.
It then creates a shm area (client_shm) and copies that buffer
to the area.  Further, another mutex (shm) is created. This
mutex is initially locked.

Next, it grabs the client lock and the server lock, and updates
client_data with the id of client_shm. Then the parent is
woken up (SIGUSR1) and the server lock is released. com_lpl()
finally goes to sleep by acquiring the shm lock.

In this situation, only the parent may run. It grabs the
server lock, reads client_data to attach client_shm and
reloads the playlist with data from client_shm. When ready,
the parent detaches client_shm, and drops the shm lock and
the server lock.

This wakes up com_lpl() which destroys client_shm, the
shm mutex and the tmp buffer.

Merge branch 'dbtool_preselect'

Merge branch 'ipc'

Conflicts:

configure.ac
error.h

Fix these conflicts.

add dbtool hooks

pre_select/post_select. Also, add a private_dbtool pointer for
each audio format to mmd.

The new ipc subsystem

Contains functions for dealing with mutexes and shared memory.

the new plm database tool

does not work yet as the ipc problems are not solved.

rename dopey to random

It isn't so dopey anymore and the upcoming plm database tool
won't be much smarter, so..

initial git commit

Let's try if this works out.