aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* database: generate GError when database is disabledMax Kellermann2011-09-133-5/+20
|
* directory: don't visit "self" in _walk()Max Kellermann2011-09-132-4/+8
| | | | Let the caller decide if the current directory should be visited.
* locate: remove unused function locate_item_new()Max Kellermann2011-09-132-17/+0
|
* directory: don't require a "directory" visitorMax Kellermann2011-09-131-6/+6
| | | | | .. to recurse. A visitor may not have one, but still be interested in a recursive search.
* database: add struct db_visitorMax Kellermann2011-09-1313-196/+363
| | | | Use this struct for db_walk().
* stored_playlist: return GError, code is playlist_resultMax Kellermann2011-09-1110-197/+320
| | | | | Improve error reporting and handling. command.c gets the new function print_error(), which sends a GError to the client.
* playlist: move enum playlist_result to playlist_error.hMax Kellermann2011-09-119-17/+50
| | | | Reduce header dependencies.
* playlist: move PLAYLIST_COMMENT to stored_playlist.cMax Kellermann2011-09-112-2/+2
| | | | Only used there.
* db_plugin: introducing a plugin API for the song databaseMax Kellermann2011-09-108-189/+529
| | | | | | | | | | | First draft, not really pluggable currently - hard-coded to use the "simple" plugin, and calls several of its internal functions. The API is very simple currently, all searches are still performed over the root "directory" object. Future changes to the API will move those search implementations into the plugin, to allow more efficient implementations, or implementations that don't have the whole tree in memory all the time.
* database: move code to db_save.cMax Kellermann2011-09-104-128/+215
|
* directory_save: pass const pointer to _save()Max Kellermann2011-09-102-4/+4
|
* tag_internal: add missing tag.h includeMax Kellermann2011-09-101-0/+2
| | | | For TAG_NUM_OF_ITEM_TYPES.
* conf: export config_param_free()Max Kellermann2011-09-092-1/+4
|
* conf: _get_next_param() returns a const pointerMax Kellermann2011-09-092-3/+3
| | | | No writers.
* database: return GError on failureMax Kellermann2011-09-094-23/+41
|
* .gitignore: add doxygen.confMax Kellermann2011-09-091-0/+1
|
* conf: add config_dup_block_path()Max Kellermann2011-09-093-14/+38
|
* utils: parsePath() returns GError on failureMax Kellermann2011-09-094-14/+30
| | | | Better error messages.
* utils: pass a const string to parsePath()Max Kellermann2011-09-092-15/+14
| | | | Remove the slash hack, allocate memory for the user name.
* utils: eliminate local variable "pos"Max Kellermann2011-09-091-3/+4
| | | | Just advance the "path" pointer.
* conf: turn config_get_path() into config_dup_path()Max Kellermann2011-09-094-47/+109
| | | | | | | | | config_get_path() was somewhat flawed, because it pretended to be a function, when it really had a side effect. The second flaw was that it did not return the parser error, instead it aborted the whole process, which is bad style. The new function returns a duplicated (modified) string that must be freed by the caller, and returns a GError on failure.
* glib_compat.h: emulate g_prefix_error()Max Kellermann2011-09-091-0/+6
|
* log: print detailed errno message on open() failureMax Kellermann2011-09-091-2/+2
|
* log: return GError on initialization failureMax Kellermann2011-09-093-11/+34
|
* conf: get_block_param() returns a const pointerMax Kellermann2011-09-093-6/+6
| | | | No caller needs to write.
* conf: move duplicate check to _read_name_value()Max Kellermann2011-09-093-17/+16
| | | | config_add_block_param() cannot fail, which makes it easier to use.
* conf: move code to config_read_name_value()Max Kellermann2011-09-091-32/+34
| | | | Reduce indent.
* conf: add gcc optimization attributesMax Kellermann2011-09-091-0/+3
|
* dbUtils: move code to db_print.cMax Kellermann2011-09-066-269/+318
|
* dbUtils: pass player_control to findAddIn()Max Kellermann2011-09-053-9/+14
| | | | Allow calling findAddIn() without a client object.
* Merge commit 'release-0.16.4'Max Kellermann2011-09-021-1/+1
|\ | | | | | | | | | | Conflicts: NEWS configure.ac
| * mpd version 0.16.4release-0.16.4Avuton Olrich2011-09-012-2/+2
| |
* | decoder/oggflac: delete this obsolete pluginMax Kellermann2011-09-016-411/+4
| | | | | | | | | | libOggFLAC has been deprecated for 5 years now, and we havn't been testing it for a long time. Let's delete it.
* | Merge branch 'v0.16.x'Max Kellermann2011-09-0115-116/+170
|\| | | | | | | | | | | Conflicts: configure.ac src/output_control.c
| * configure.ac: fail if Vorbis was enabled explicitly, but not foundMax Kellermann2011-09-011-25/+18
| | | | | | | | .. and a minor tweak for libFLAC+libogg detection.
| * configure.ac: fail if FLAC was enabled explicitly, but not foundMax Kellermann2011-09-011-6/+8
| |
| * output/osx: don't drain the buffer when closingMax Kellermann2011-09-012-6/+1
| | | | | | | | Eliminate an unnecessary source of deadlocks.
| * output/osx: signal the GCond while mutex is lockedMax Kellermann2011-09-011-1/+1
| |
| * configure.ac: auto-detect libmad without pkg-configMax Kellermann2011-09-012-1/+3
| | | | | | | | | | The pkg-config file was added by the Debian package maintainers, and unfortunately, the rest of the world doesn't have it.
| * configure.ac: fail if libid3tag was enabled explicitly, but not foundMax Kellermann2011-09-012-10/+20
| | | | | | | | | | Add M4 function MPD_AUTO_PKG_LIB for pkg-config with AC_CHECK_LIB fallback.
| * configure.ac: use MPD_AUTO_PKG to detect avahiMax Kellermann2011-09-012-14/+18
| | | | | | | | | | | | Don't abort the configure script when avahi could not be auto-detected. It previously did, because there was no custom "fail" action for PKG_CHECK_MODULES.
| * Makefile.am: use AVAHI_CFLAGS, AVAHI_LIBSMax Kellermann2011-09-012-2/+3
| | | | | | | | Don't add those to MPD_CFLAGS / MPD_LIBS.
| * output_all: move _lock_signal() to output_control.cMax Kellermann2011-09-013-21/+23
| | | | | | | | Better name, better documentation.
| * output_thread: reimplement CANCEL synchronizationMax Kellermann2011-09-017-14/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The output thread could hang indefinitely after finishing CANCEL, because it could have missed the signal while the output was not unlocked in ao_command_finished(). This patch removes the wait() call after CANCEL, and adds the flag "allow_play" instead. While this flag is set, playback is skipped. With this flag, there will not be any excess wait() call after the pipe has been cleared. This patch fixes a bug that causes mpd to discontinue playback after seeking, due to the race condition described above.
| * output/pulse: reset callbacks before closing stream/contextMax Kellermann2011-08-311-0/+10
| | | | | | | | | | Fixes assertion failure when a stream callback is invoked too late after a format change.
| * output/pulse: add function _delete_stream()Max Kellermann2011-08-311-5/+16
| | | | | | | | Merge common code.
| * output/pulse: use _delete_context()Max Kellermann2011-08-311-16/+15
| | | | | | | | Eliminate duplicate code.
| * doc/developer.xml: change the coing style example return type to intJonathan Neuschäfer2011-08-271-1/+1
| |
| * scripts/makedist.sh: fix test usageJonathan Neuschäfer2011-08-271-2/+2
| | | | | | | | Checkbashisms (part of the Debian devscripts) pionted this out.
| * input/curl: implement a hard-coded timeout of 10 secondsMax Kellermann2011-08-262-0/+4
| | | | | | | | | | Be sure to stop the operation at some point when the server isn't responding.