A future patch will implement a 24 bit resampler.  To unify code, move
code which can be shared to a separate function.

Copy from source to destination buffer directly, don't use the
temporary variables "lsample" and "rsample".

Added assertions which ensure that there are no partial samples in the
source buffer.

Use sizeof(sample) instead of hard-coding "2".  Although we're in 16
bit right now, this will make code sharing easier when we support
other sample sizes.

Separate the resampling code from the rest of pcm_utils.c.  Create two
sub-libraries: pcm_resample_libsamplerate.c and
pcm_resample_fallback.c.

Due to a logic error, no value was valid for the boolean value
parser.  Replace "||" with "&&".

libmad produces samples of more than 24 bit.  Rounding that down to 16
bits using dithering makes those people lose quality who have a 24 bit
capable sound device.  Send 24 bit PCM data, and let the receiver
decide whether to apply 16 bit dithering.

We are going to convert the code to 24 bit; don't hard-code a sample
size of 2 bytes.

Copied and adapted code from the mp3 decoder plugin.  This library now
replaces the old and low-quality function pcm_convert_24_to_16().

I added 24 bit support a while ago, but it wasn't possible to force 24
bit output.  Add 24 and 8 bit to the list of allowed sample sizes.
Although 8 bit audio isn't as widely used as 24 bit, there is no
reason to exclude it.

Splitting a frame between two buffer chunks causes distortion in the
output.  MPD used to assume that the chunk size 1020 would never cause
splitted frames, but that isn't the case for 24 bit stereo (127.5
frames), and even less for files with even more channels.

Don't attempt to map paths which are already absolute with
map_song_fs(): check with song_in_database() instead of
song_is_file().

The "rm" command did not send notifications to idle clients.  Add it.

The function deletePlaylist() shouldn't be in playlist.c.

Don't use our deprecated linked list library, use GLib's GPtrArray
instead.

The "long" result of strtol() was implicitly casted down to a 32 bit
integer.  Add some range checking instead.

Many command arguments must not be negative; add a separate
parser/checker function for that.  For the same reason, add
check_bool().  This eliminates two strange special cases handlers from
check_int().

Pass index arguments as unsigned integers.  They must not be negative,
and even if some caller accidently passes -1, it won't pass the bound
checks (since it's now 2**32-1).

There are some integers which have a "magic" -1 value which means
"undefined" or "nothing".  All others can be converted to unsigned,
since they must not contain a negative number.

Instead of returning 0 for success and -1 for failure, return true or
false.  This seems more natural.

Also add names for "error" and "ok".  I don't like passing anonymous
integer codes around.

This is not yet complete: lots of functions (e.g. in playlist.c)
follow the same convention of -1/0, and these have to be adapted, too.

http://xkcd.com/292/

Eliminate CamelCase in all public and static functions.

The typedef CommandHandlerFunction is only used once.  Move its type
into the command struct.

Send last modification timestamps to the client.  This allows the
client to see when another client modifies a stored playlist.

"listplaylists" returns a list of all stored playlists.  This command
seems more elaborate than listing them below "lsinfo".

The function print_spl_list() replaces the old function lsPlaylists()
from ls.c.

spl_list() provides an interface for enumerating all stored playlists.
This separates the internal playlist logic from the protocol specific
function lsPlaylists().

The return value of spl_append_uri() was somewhat buggy: some branches
returned ACK_* values, and some an enum playlist_result.  Unify this.

Rename addToStoredPlaylist() to spl_append_uri(), and remove the
clearStoredPlaylist() macro.

The two functions clearStoredPlaylist() and addToStoredPlaylist()
don't belong into playlist.c.  clearStoredPlaylist() was a wrapper for
spl_clear(), and is converted into a CPP macro for now.

Renamed all public functions, prefix is "spl_".

No CamelCase in file names.

The list of commands is known at compile time.  Instead of creating a
linked list on startup, we can just register all commands in a static
sorted array.

The command pointers which are passed around aren't being modified -
in fact, no command pointer must be modified once it has been added to
the commandList.

No CamelCase and no struct typedefs.

Instead of manually calling memset(0) on the pcm_convert_state struct,
client code should use a library function from pcm_utils.c.  This way,
we can change the semantics of the struct easily.

Renamed all functions which were still in CamelCase.

No CamelCase, and a struct instead of a typedef.