Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | output_thread: fix double lock | Max Kellermann | 2011-01-07 | 2 | -3/+1 |
| | | | | | | | | During the whole output thread, the audio_output object is locked, and it is only unlocked while waiting for the GCond and while running a plugin method. The error handler in ao_play_chunk() attempted to lock the object again, which was code from MPD 0.15.x which should have been removed a long time ago. | ||||
* | player_thread: fix assertion failure due to early seek | Max Kellermann | 2011-01-07 | 2 | -0/+5 |
| | | | | | | | Until the decoder plugin has called decoder_initialized(), the player may not submit seek commands. This however could occur with a slow decoder and a CUE file with a virtual song offset. This patch adds another check. | ||||
* | player_thread: make variables more local | Max Kellermann | 2011-01-07 | 1 | -36/+19 |
| | |||||
* | output/osx: fix up audio format first, then apply it to device | Max Kellermann | 2011-01-07 | 2 | -9/+11 |
| | | | | | | | | | This is a MPD 0.16 regression: when playing a 24 bit file, the switch to 16 bit was made only partially, after mBytesPerPacket and mBytesPerFrame had already been applied. That means mBytesPerFrame referred to 24 bit, and mBitsPerChannel referred to 16 bits. Of course, that cannot work. | ||||
* | configure.ac: eliminate bashism "echo -n" | Max Kellermann | 2011-01-07 | 2 | -20/+20 |
| | | | | Use "printf" instead. | ||||
* | configure.ac: avoid GNU extension in "expr match" call | Max Kellermann | 2011-01-07 | 1 | -1/+1 |
| | |||||
* | configure.ac: disable the FFADO plugin by default | Max Kellermann | 2010-12-22 | 2 | -1/+3 |
| | | | | It is known to crash instantly. | ||||
* | add void casts to suppress "result unused" warnings (clang) | Max Kellermann | 2010-12-21 | 3 | -3/+4 |
| | |||||
* | decoder/mad: work around build failure on Solaris | Alex Viskovatoff | 2010-12-21 | 2 | -2/+4 |
| | | | | | | | | Rename the "version" struct, because it seems to be a reserved name on Solaris: "src/decoder/mad_decoder_plugin.c", line 550: (enum) tag redeclared: version cc: acomp failed for src/decoder/mad_decoder_plugin.c | ||||
* | output/solaris: add missing parameter to open_cloexec() call | Alex Viskovatoff | 2010-12-21 | 2 | -1/+3 |
| | |||||
* | audio_check: fix parameter in prototype | Alex Viskovatoff | 2010-12-21 | 2 | -1/+2 |
| | |||||
* | Modify version string to post-release version 0.16.1~git | Max Kellermann | 2010-12-21 | 2 | -1/+4 |
| | |||||
* | mpd version 0.16release-0.16 | Avuton Olrich | 2010-12-11 | 2 | -2/+2 |
| | |||||
* | Merge branch 'master' of git://git.musicpd.org/avuton/mpd | Max Kellermann | 2010-12-07 | 0 | -0/+0 |
|\ | |||||
| * | Modify version string to post-release version 0.16~git | Avuton Olrich | 2010-11-08 | 1 | -1/+1 |
| | | |||||
| * | mpd version 0.16~alpha4release-0.16_alpha4 | Avuton Olrich | 2010-11-08 | 1 | -1/+1 |
| | | |||||
* | | tag_id3: support multiple values | Max Kellermann | 2010-12-07 | 2 | -10/+37 |
| | | | | | | | | | | Loop over all frames with a specific id, and import all of them - not just the first one (index 0). | ||||
* | | command: don't error when sticker list is run on song with no stickers | Anton Khirnov | 2010-12-07 | 1 | -7/+3 |
| | | | | | | | | | | | | this is inconsistent with other commands (e.g. find) and seems wrong -- a song with no stickers attached is a perfectly valid state and an empty list of stickers is also perfectly valid. | ||||
* | | tag_ape: support multiple values | Max Kellermann | 2010-11-24 | 2 | -1/+16 |
| | | | | | | | | One APE tag may contain more than one value, separated by null bytes. | ||||
* | | decoder_thread: load APE replay gain from music files | Max Kellermann | 2010-11-18 | 1 | -0/+15 |
| | | |||||
* | | replay_gain_ape: parse replay gain from APE tags | Max Kellermann | 2010-11-18 | 4 | -0/+113 |
| | | | | | | | | Based on the APE reader. | ||||
* | | tag_ape: move code to ape.c | Max Kellermann | 2010-11-18 | 4 | -93/+176 |
| | | | | | | | | Generic library for scanning APE tags. Eliminated one "goto"! | ||||
* | | timer: fix integer overflow in timer_delay() | Max Kellermann | 2010-11-18 | 1 | -1/+1 |
|/ | | | | | | | Fixes a regression: for output_plugin.delay(), we added a method to the timer class which returns the delay in milliseconds. This fails to detect negative values, because the unsigned integer is divided by 1000, and then casted to signed. | ||||
* | playlist_song: calculate duration of last CUE track | Max Kellermann | 2010-11-08 | 1 | -0/+8 |
| | |||||
* | Merge release 0.15.15 from branch 'v0.15.x' | Max Kellermann | 2010-11-08 | 3 | -2/+9 |
|\ | | | | | | | | | | | Conflicts: NEWS configure.ac | ||||
| * | mpd version 0.15.15release-0.15.15 | Max Kellermann | 2010-11-08 | 2 | -2/+2 |
| | | |||||
| * | output/shout: artist comes first in stream title | Max Kellermann | 2010-11-08 | 2 | -1/+3 |
| | | | | | | | | | | | | After popular demand, I've switched the order of "artist" and "title" in the stream title. There is no standard, and there is no reliable way to parse those from the stream title. | ||||
| * | input/rewind: fix assertion failure | Max Kellermann | 2010-11-08 | 2 | -1/+3 |
| | | | | | | | | | | The assertion added in MPD 0.15.14 was too much, it failed when the MIME type of a stream was NULL. | ||||
| * | Modify version string to post-release version 0.15.15~git | Avuton Olrich | 2010-11-06 | 2 | -1/+4 |
| | | |||||
* | | update_walk: explicitly check for permission problems | Max Kellermann | 2010-11-08 | 1 | -0/+33 |
| | | | | | | | | | | | | Call access() and print an extra error message when EACCES is returned. Hopefully this will reduce the number of support requests due to wrong file permissions. | ||||
* | | Modify version string to post-release version 0.16~git | Avuton Olrich | 2010-11-07 | 1 | -1/+1 |
| | | |||||
* | | mpd version 0.16~alpha3release-0.16_alpha3 | Avuton Olrich | 2010-11-07 | 1 | -1/+1 |
| | | |||||
* | | Merge release 0.15.14 from branch 'v0.15.x' | Max Kellermann | 2010-11-07 | 11 | -45/+109 |
|\| | | | | | | | | | | | | | | | | | | | | | | | Conflicts: NEWS configure.ac src/decoder_control.c src/decoder_control.h src/input/rewind_input_plugin.c src/output_control.c src/output_thread.c src/player_thread.c | ||||
| * | mpd version 0.15.14release-0.15.14 | Avuton Olrich | 2010-11-06 | 2 | -2/+2 |
| | | |||||
| * | player_thread: fix assertion failure due to wrong music pipe on seek | Max Kellermann | 2010-11-05 | 2 | -0/+9 |
| | | | | | | | | | | | | | | | | | | When one song is played twice, and the decoder is working on the second "instance", but the first should be seeked, the check in player_seek_decoder() may assume that it can reuse the decoder without exchanging pipes. The last thing was the mistake: the pipe pointer was different, which led to an assertion failure. This patch adds another check which exchanges the player pipe. | ||||
| * | decoder_control: pass music_pipe to dc_start() | Max Kellermann | 2010-11-05 | 3 | -12/+18 |
| | | | | | | | | More abstraction for decoder_control.pipe. | ||||
| * | player_thread: add helper function player_dc_at_next_song() | Max Kellermann | 2010-11-05 | 1 | -6/+33 |
| | | | | | | | | Some abstraction for decoder_control.pipe access. | ||||
| * | pipe: add helper function music_pipe_empty() | Max Kellermann | 2010-11-05 | 3 | -4/+10 |
| | | |||||
| * | input/rewind: remove redundant NULL check before g_free() call | Max Kellermann | 2010-11-05 | 1 | -2/+1 |
| | | |||||
| * | input/rewind: add two assertions | Max Kellermann | 2010-11-05 | 1 | -0/+3 |
| | | |||||
| * | input/rewind: fix double free bug | Max Kellermann | 2010-11-05 | 2 | -0/+3 |
| | | | | | | | | | | Duplicate the "mime" attribute of the inner input_stream object, instead of copying the pointer. | ||||
| * | decoder/{mp4ff,ffmpeg}: add extension ".m4b" (audio book) | Max Kellermann | 2010-11-05 | 3 | -2/+13 |
| | | | | | | | | Same as ".m4a". | ||||
| * | output_thread: fix assertion failure due to race condition in OPEN | Max Kellermann | 2010-11-04 | 2 | -2/+10 |
| | | | | | | | | | | | | Change the assertion on "fail_timer==NULL" in OPEN to a runtime check. This assertion crashed when the output thread failed while the player thread was calling audio_output_open(). | ||||
| * | output_internal: protect attribute "fail_timer" with mutex | Max Kellermann | 2010-11-04 | 2 | -1/+7 |
| | | |||||
| * | output_control: lock object in audio_output_open() | Max Kellermann | 2010-11-04 | 1 | -6/+19 |
| | | | | | | | | Protect the attributes "open" and "fail_timer". | ||||
| * | output_control: lock object in audio_output_close() | Max Kellermann | 2010-11-04 | 1 | -2/+28 |
| | | | | | | | | Protect the attributes "open" and "fail_timer". | ||||
| * | Modify version string to post-release version 0.15.14~git | Avuton Olrich | 2010-10-10 | 2 | -1/+4 |
| | | |||||
* | | output/alsa: dump buffer and period limits | Max Kellermann | 2010-11-05 | 1 | -0/+20 |
| | | |||||
* | | output/httpd: implement delay() | Max Kellermann | 2010-11-05 | 2 | -3/+12 |
| | | |||||
* | | output/shout: implement delay() | Max Kellermann | 2010-11-05 | 2 | -8/+14 |
| | | | | | | | | | | This makes the plugin more responsive to control commands, because it will listen to control events while waiting. |