This line was missing in the reverse_endian patch, and led to
undefined values and crashes in that attribute.

Changed function to first close standard input (this may
fail but we don't care) and then try to open /dev/null (this
may fail but it shouldn't on Unix platforms plus we don't
know what to do in such case anyways).  Since standard input
has the "zeroth" descriptor number next "open" will use it.

Since there is no "/dev/null" on Windows (It's not even
a valid path!) the second step is skipped if WIN32 is
defined.

As a final touch, since the function consists of merely two
function calls it has been moved to header file and declared
static inline.

[mk: un-inline daemonize_close_stdin()]

On FreeBSD, "stderr" is a macro, and using this name for a struct
member breaks the build.

[mk: renamed _stderr to log_stderr]

Conflicts:

	Makefile.am

Make "enum tag_type" the array index, and convert apeItems to a sparse
array.

Improve code readability.

Don't initialize those arrays each time tag_ape_load() is called.

It makes no difference right now, but we're about to add an endianness
flag and will want to make sure it's correctly initialised every time.

Several functions work with the wrong return type, this patch fixes
them.

The new code is more robust and more flexible.  It provides detailed
error information in GError objects.

On Windows only daemonize_close_stdin() function does
something.  Other functions are either empty or generate an
error.  Those have been moved to header file and declared
static inline so compiler can remove the call all together.

The functions were not "const" (they examined values pointed
by arguments passed to them, quoting gcc's doc: "Note that
a function that has pointer arguments and examines the data
pointed to must _not_ be declared 'const'.") but rather
"pure" and still not all of them.

Note also, that even some of the functions declared "pure"
are not pure, however, due to reasons stated in source code
the attribute has been kept.

The "group" configuration option is similar to "user" as it
sets user set what group MPD shall run as.  With "user"
option, MPD changed GID to the GID of the user, however,
more control could be desired.

Moreover, the patch changes the way of checking whether no
setuid(2)/setgid(2) is required -- previously user names
were compered, now UID and GIDs are compered (ie. the one we
already have (getuid(2)/getgid(2)) with the one we want to
change to).

The GLib functions are more portable.

Added another glue function in main().

The glue_*() functions act as a glue between MPD's main() function and
its libraries.  They handle disabled features, and pass validated
configuration options.

No CamelCase.

Since version 0.14, MPD has been logging to standard error instead of
standard output.  The option name should reflect that.  The old option
continues to work, we will remove it in a future MPD release.

Renamed type, variables and functions.

Add the "const" attribute to functions when their return value only
depends on parameters.  This allows gcc to eliminate some function
calls.

If the encoder plugin API is disabled at compile time, don't compile
test/run_encoder.c.

Parse the state file line by line, let each subsystem probe a line.
Only the playlist_state code gets the FILE pointer to read the
following lines.

Return early from the destructor function when there is no configured
state file.  Don't check the timer, don't call g_free(NULL).

Print "Loading" instead of "Saving" in state_file_read().  Added debug
message to state_file_write().

This encoder plugin is a replacement for the LAME encoder plugin for
those who prefer a "free" (non-patent encumbered) encoder library.
Most of the plugin source code is copied from the LAME encoder plugin,
since the LAME and TwoLAME APIs are nearly the same.

Fail when a feature is enabled, but the library is not found.

Don't append these to MPD_CFLAGS and MPD_LIBS.

Those were only wrappers for playlist_state_{save,restore}().  Since
sf_callbacks has been removed, we can call the latter functions
directly.

There are very few callbacks, and they are not meant to be pluggable.
Let's eliminate the array and call the load/save functions manually.