aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* volume: removed unused macrosMax Kellermann2009-07-031-2/+0
| | | | Removed VOLUME_MIXER_OSS and VOLUME_MIXER_ALSA.
* filter: added "volume" pluginMax Kellermann2009-07-033-0/+138
| | | | | | | | | | The "volume" filter plugin will replace the current software volume code. One "volume" filter may be attached to each output device. This will allow the user to use hardware mixers for some devices, and software mixers for other devices at the same time. Currently, neither the filter API nor the "volume" plugin is integrated into MPD.
* filter: added new plugin API for filtersMax Kellermann2009-07-036-0/+455
| | | | | The filter API allows us to implement software volume as a pluggable filter, and we will be able to integrate libraries like SoX.
* conf: added config_quark()Max Kellermann2009-07-031-0/+10
| | | | | This GQuark will be used for GErrors related to malformed configuration.
* 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-261-5/+9
| | | | | | | | | | | | | 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.
* ffmpeg: support multiple tagsMax Kellermann2009-06-251-2/+3
| | | | Call av_metadata_get() in a loop.
* conf: log unused/unknown block parametersMax Kellermann2009-06-253-0/+37
|
* 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-253-89/+2
| | | | | 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-251-0/+2
|
* Preamp for missing replay-gainDaniel Seuthe2009-06-254-11/+41
|
* 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-101-3/+25
| | | | | | | | | 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.
* 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-034-21/+22
| | | | Renamed all remaining CamelCase functions.
* 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.
* 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.
* music_chunk: added more audio_format_valid() assertionsMax Kellermann2009-05-111-0/+2
| | | | Check the validity of the audio_format during write operations.
* music_pipe: added assertion on chunk->audio_formatMax Kellermann2009-05-111-0/+1
| | | | Always assert that the audio format of the new chunk is valid.
* latest git, httpd_output_plugin compile fix, mac os xPatrik Weiskircher2009-05-101-0/+4
| | | | | | | | | | Hello, While compiling latest git I've received a compile error in the httpd_output_plugin. Small patch attached. Patrik
* pcm_resample: fixed typo in libsamplerate runtime checkMax Kellermann2009-05-071-1/+1
| | | | The string comparison should be "!= 0", not "== 0". Ouch.
* player_control: assert that pc.next_song is NULL after seekingMax Kellermann2009-05-061-0/+2
| | | | | The player thread must reset pc.next_song after seeking, even if that operation has failed. This patch adds an assertion.
* player_control: removed pc.error check from pc_seek()Max Kellermann2009-05-061-6/+3
| | | | | | The only pc_seek() caller clears the error, rendering the check useless. Even if the previous PLAY command resulted in a player error, this check is not very useful.
* player_control: renamed playerSeek(), return boolMax Kellermann2009-05-063-9/+15
| | | | | Renamed playerSeek() to pc_seek() to get rid of CamelCase. Convert the return value to bool.
* tag_id3: added support for the UFID frameMax Kellermann2009-05-061-0/+40
| | | | | If the UFID frame's owner is "http://musicbrainz.org", assume its value is the MusicBrainz track id.
* httpd_output: save the page generated by encoder_tag()Max Kellermann2009-05-051-0/+21
| | | | | | | | Flush the encoder before calling encoder_tag(). The first page generated by the encoder after sending the tag will be the new "header" page, which is sent to all HTTP clients when they connect. This is a little bit specific to the vorbis encoder, but there are no other encoders which support tags (yet).
* httpd_output: moved code to httpd_output_encoder_to_clients()Max Kellermann2009-05-051-14/+32
| | | | | | Moved some code from httpd_output_encode_and_play() into separate functions httpd_output_broadcast_page() and httpd_output_encoder_to_clients().
* httpd_output: disable Icy-Metadata when encoder supports tagsMax Kellermann2009-05-053-18/+35
| | | | | | There's no reason to send both encoder tags and Icy-Metadata to the client. Let's disable Icy-Metadata when the encoder supports embedded tags.
* vorbis_encoder: start a new stream in tag()Max Kellermann2009-05-051-6/+9
| | | | | | | When a new tag is set, end the current stream and begin a new one. Use vorbis_analysis_headerout() to write a full ogg header. This fixes a problem with icecast: after a song change in MPD, icecast stops forwarding ogg packets to its clients.