aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* playlist_state: don't save "current" song when none is setMax Kellermann2009-06-291-2/+4
| | | | | | | | | This patch fixes an assertion failure: Assertion `order < queue->length' failed. This happens when the state file is saved, when there is no "current" song: current==-1, and queue_order_to_position(-1) is called.
* playlist_state: save state when stoppedChristopher Zimmerman2009-06-262-5/+10
| | | | | | | | | | | | | At the moment mpd doesn't store or restore the current track to/from its state file when the daemon is stopped/started while in 'stopped' state. I believe the preferred behaviour would be to store and restore the current track even when the daemon is in stopped state when shutting down. I made a small patch to adapt this behaviour. If you believe this is not the preferred behaviour, maybe this should be realized as a configuration option. I'm not sure how to do this, but made a small comment, where one would have to put the option.
* autogen.sh: don't check for autoconf <2.60Max Kellermann2009-06-251-1/+1
| | | | | configure.ac refuses to run with autoconf older than 2.60, don't bother to test for those versions in autogen.sh.
* Makefile.am: require automake 1.10Max Kellermann2009-06-253-3/+3
| | | | | Refuse to build with automake 1.9. 1.9 is quite old already, and I'm too lazy to test with ancient versions.
* ffmpeg: support multiple tagsMax Kellermann2009-06-252-2/+5
| | | | Call av_metadata_get() in a loop.
* conf: log unused/unknown block parametersMax Kellermann2009-06-254-0/+38
|
* conf: added "used" flag to struct block_paramMax Kellermann2009-06-252-1/+16
|
* conf: detect duplicate parameters in config_add_block_param()Max Kellermann2009-06-251-9/+9
| | | | | Moved the check from config_get_block_param(). Detect the duplicate parameter when it's added, not when it's queried.
* conf: make get_bool() return a boolMax Kellermann2009-06-252-22/+18
| | | | | | | Instead of returning an artificial three-state integer, return a "success" value and put the boolean value into a "bool" pointer. That's a little bit more overhead, but an API which looks more natural.
* conf: moved code from get_bool() to string_array_contains()Max Kellermann2009-06-251-9/+16
|
* conf: register configuration options staticallyMax Kellermann2009-06-251-94/+57
| | | | | Initialize the config_entries array at compile time. This is not only faster, but also smaller.
* conf: removed config_add_param()Max Kellermann2009-06-252-16/+0
| | | | This function is unused.
* conf: replace "mask" bit field with two "bool" variablesMax Kellermann2009-06-251-12/+6
| | | | Due to padding, this takes the same amount of memory.
* volume: removed support for legacy mixer configurationMax Kellermann2009-06-254-89/+4
| | | | | The top-level "mixer_device" and "mixer_control" options have been deprecated by MPD 0.15, and it's safe to remove them in MPD 0.16.
* command: added new "status" line with more precise "elapsed time"Max Kellermann2009-06-253-0/+14
|
* Preamp for missing replay-gainDaniel Seuthe2009-06-254-11/+41
|
* Modify version string to post-release version 0.16~gitAvuton Olrich2009-06-242-1/+4
|
* mpd version 0.15release-0.15Avuton Olrich2009-06-232-2/+2
|
* decoder_thread: reopen the stream after file_decode() has failedMax Kellermann2009-06-191-0/+11
| | | | | | | | | When decoding a local file, the decoder thread tries to run all matching decoders, until one succeeds. Both file_decode() and stream_decode() can decode a stream, but MPD closes the stream before calling file_decode(). Problem is: when this decoder fails, and the next's stream_decode() method is invoked, the input_stream is still closed. This patch reopens it.
* listen: bind() failure on secondary address is non-fatalMax Kellermann2009-06-102-3/+26
| | | | | | | | | Several users had problems with binding MPD to "localhost". The cause was duplicate /etc/hosts entries: the resolver library returns 127.0.0.1 twice, and of course, MPD attempts to bind to "both" of them. This patch makes failures non-fatal, given that at least one address was bound successfully. This is a workaround; users should rather fix their /etc/hosts file.
* listen: renamed "error" to "error_r" in listen_add_host()Max Kellermann2009-06-101-4/+4
|
* listen: print debug message before bind()Max Kellermann2009-06-101-0/+7
| | | | Dump each socket address before binding to it.
* doc: refer to command_list_ok_begin, not command_list_beginMax Kellermann2009-06-091-1/+1
| | | | list_OK is returned only after command_list_ok_begin.
* doc: fixed a typo in the protocol documentationMax Kellermann2009-06-091-1/+1
|
* ffmpeg: moved code to ffmpeg_copy_metadata()Max Kellermann2009-06-081-33/+18
|
* ffmpeg: removed "new metadata api" warningMax Kellermann2009-06-081-2/+0
| | | | | This warning is useless. I assume the author added it for debugging purposes.
* client: check "expired" flag again in client_write_output()Max Kellermann2009-06-041-0/+3
| | | | | | | When client_defer_output() aborts the connection to the client, client_write_output() called client_write_deferred() anyway. This caused an assertion failure. Fix it by checking for the "expired" flag again after client_defer_output() returns.
* client: added assertions on channel!=NULLMax Kellermann2009-06-041-0/+10
| | | | | | I'm hunting down a bug where client->channel==NULL during I/O operations. These new assertions help avoid this kind of bug in the future.
* decoder_plugin: corrected outdated API documentationMax Kellermann2009-06-041-21/+22
|
* conf: make config_param.num_block_params unsignedMax Kellermann2009-06-032-5/+3
|
* conf: eliminated CamelCaseMax Kellermann2009-06-035-23/+24
| | | | Renamed all remaining CamelCase functions.
* doc: mark the "User's Manual" as incompleteMax Kellermann2009-06-021-0/+2
|
* doc: added Developer's ManualMax Kellermann2009-06-023-1/+111
| | | | Just a start...
* doc: added "since MPD version" to commandsMax Kellermann2009-06-021-4/+17
| | | | Document which commands were introduced after MPD 0.13.
* player_thread: check pipe size again before exitingMax Kellermann2009-06-021-1/+5
| | | | | | When the decoder is finished, break out of the player loop only after another player.pipe check. We did check the pipe size a few lines above, but that check was kind of racy.
* player_thread: don't leak empty music_chunksMax Kellermann2009-06-021-1/+3
| | | | | | | When a music_chunk only contains a tag but no PCM data, play_chunk() returns true without freeing the chunk. The caller now assumes that the chunk is moved into some music_pipe and does not bother to free it either.
* player_thread: reinitialize music_buffer in !NDEBUGMax Kellermann2009-05-291-0/+9
| | | | | | To check for leaked music_chunk objects, free the music buffer on CLOSE_AUDIO. This invokes an assertion check which ensures that all chunks have been returned to the buffer.
* output_all: explicitly return "true" from audio_output_all_play()Max Kellermann2009-05-291-1/+1
| | | | | | Instead of returning the local variable "ret" which is always true at this point, hard-code the "true" return value, because that might be more readable.
* input/file: log message on errorMathieu Rochette2009-05-281-0/+2
| | | | | | | | If a file is removed the library, next time mpd will try to play it it will result in an error 'ERROR: problems decoding some/file.ogg'. Nothing is written in log files (verbose mode or not) [mk: append strerror(errno)]
* volume: changed "default" to "DISABLED" in switch statementMax Kellermann2009-05-281-1/+4
| | | | | Make gcc warn us if we add a new mixer type, and forget to add a new "case" line.
* added doc/sticker to .gitignoreMax Kellermann2009-05-281-0/+1
|
* volume: converted volume_mixer_type to an enumMax Kellermann2009-05-281-5/+6
| | | | Don't use CPP macros when you can use C enums.
* volume: removed unused macro "VOLUME_MIXER_SOFTWARE_DEFAULT"Max Kellermann2009-05-281-1/+0
|
* playlist: reset "current song" when playlist endsMax Kellermann2009-05-281-0/+3
| | | | | | Commit f78cddb4 introduced a regression: when the playlist reached its end, MPD did not reset the "current song" pointer anymore after stop. Add a "current = -1" code line.
* Incorrect identifying of --with-tremor option in configure.acVictor A. Safronov2009-05-261-3/+9
| | | | [mk: moved check out of the AC_ARG_WITH block]
* configure.ac: use $FAAD_LIBS in libmp4ff checkMax Kellermann2009-05-261-1/+1
| | | | | Append $FAAD_LIBS to $LIBS when detecting libmp4ff. $FAAD_LIBS may contain an important -L flag.
* doc: fixed DocBook markup in doc/sticker.xmlMax Kellermann2009-05-152-43/+56
| | | | | The old sticker.xml used elements which are not valid in DocBook. Now that the file is valid, let's add it to $(DOCBOOK_FILES).
* Makefile.am: added doc/doxygen.conf to $(EXTRA_DIST)Max Kellermann2009-05-151-1/+1
|
* configure.ac: removed the --disable-lametest optionMax Kellermann2009-05-151-9/+0
| | | | | | | | The M4 function AM_PATH_LAME (m4/lame.m4) defined a configure flag named "--disable-lametest". This is redundant with configure.ac's --disable-lame-encoder, and specifying both options may break the build. Since AM_PATH_LAME is only called when the encoder plugin is enabled, we can safely remove that --disable-lametest option.
* music_chunk: added more audio_format_valid() assertionsMax Kellermann2009-05-111-0/+2
| | | | Check the validity of the audio_format during write operations.