aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* decoer/dsf: fix endless loop on malformed fileMax Kellermann2014-08-212-1/+2
| | | | | | | When the data chunk size is not a multiple of the frame size, the last partial frame lead to an endless loop. We fix this by checking chunk_sze>=frame instead of chunk_sze>0. This way, the partial frame is simply skipped.
* util: Fix header for strcasecmpFrançois Revol2014-08-161-1/+1
| | | | | | | | | | | | | | | | | According to POSIX and both OSX and Linux manpages, strcasecmp comes from strings.h, not string.h. Most OSes also have them available in string.h, but we just fixed the headers on Haiku and it now only provides them in strings.h. We might want to fall back to string.h for other OSes though... cf. http://pubs.opengroup.org/onlinepubs/009695399/functions/strcasecmp.html http://linux.die.net/man/3/strcasecmp https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man3/strcasecmp.3.html
* decoder/ffmpeg: use avcodec_descriptor_get() to determine codec nameMax Kellermann2014-08-132-0/+11
| | | | | | In version 11, both ffmpeg and libav deprecate AVCodecContext::codec_name. The function avcodec_descriptor_get() has been introduced long ago.
* event/BufferedSocket: fix inversed buffer checkMax Kellermann2014-08-071-1/+1
| | | | | This was broken by commit 84d20d9e, which deleted the "!" from the check.
* Makefile.am: fix dependencies for win32François Revol2014-08-021-1/+1
| | | | | | | | | | | | | | It happened to me when doing the Haiku port, src/mpd failed to be relinked properly when editing source files, and likely also happens on win32, although I didn't try this change. When building for windows, src_mpd_DEPENDENCIES is overriden. Automake then disables the default version which contains all the static libraries. In Makefile.in: @HAVE_WINDOWS_FALSE@src_mpd_DEPENDENCIES = libmpd.a \ Instead we use EXTRA_src_mpd_DEPENDENCIES which is meant for this.
* Win32Main: move to win32/Max Kellermann2014-08-026-9/+9
|
* configure.ac: prepare for 0.18.13Max Kellermann2014-08-022-1/+3
|
* release v0.18.12Max Kellermann2014-07-301-1/+1
|
* decoder/dsdiff: simplify dsdlib_skip() callMax Kellermann2014-07-121-4/+3
|
* decoder/dsdiff: simplify loop condition, merge branchesMax Kellermann2014-07-121-7/+2
|
* decoder/dsdiff: ignore garbage null byte at end of fileMax Kellermann2014-07-121-1/+1
| | | | | Failure to read another chunk header is not fatal. Continue to read metadata.
* decoder/dsdiff: fix metadata parser bug (uninitialized variables)Max Kellermann2014-07-122-1/+5
|
* decoder/faad: estimate song duration for remote filesMax Kellermann2014-07-122-0/+23
| | | | | | | | Previously, MPD tried to slurp the whole song file, count the number of frames and calculate the song duration from that. That however is extremely expensive for remote files, and will delay playback for a long time. Workaround: check only the first 128 frames and try to extrapolate from here. Fixes Mantis ticket 0004035.
* DecoderBuffer: add method _available()Max Kellermann2014-07-122-0/+13
|
* DecoderBuffer: add method _get_stream()Max Kellermann2014-07-122-0/+10
|
* decoder/faad: bail out early if sample rate is invalidMax Kellermann2014-07-121-0/+2
|
* decoder/faad: use adts_check_frame() in faad_song_duration()Max Kellermann2014-07-121-2/+1
| | | | Eliminate more duplicate code.
* decoder/faad: test "seekable" after ADTS frame checkMax Kellermann2014-07-121-1/+5
| | | | | Don't bother to check for ADIF just because the stream is not seekable.
* decoder/faad: move code to faad_decoder_new()Max Kellermann2014-07-121-14/+17
| | | | Merge some duplicate code.
* decoder/faad: remove unnecessary readMax Kellermann2014-07-121-2/+0
| | | | Eliminate some overhead when the caller doesn't need the buffer.
* DecoderBuffer: add "pure" attributesMax Kellermann2014-07-121-0/+4
|
* DecoderBuffer: add method _clear()Max Kellermann2014-07-123-8/+12
|
* decoder/faad: check sample_rate, not frames_per_secondMax Kellermann2014-07-111-2/+4
| | | | Checking the integer is faster, easier and more reliable.
* decoder/faad: make variables more localMax Kellermann2014-07-111-71/+49
|
* decoder/faad: use MAX_CHANNELSMax Kellermann2014-07-111-4/+2
| | | | .. instead of declaring a new constant.
* decoder/sndfile: use decoder_read_full()Max Kellermann2014-07-111-16/+6
| | | | | Replaces the loop in sndfile_vio_read(), eliminating duplicate and fragile code.
* decoder/sndfile: use decoder_read()Max Kellermann2014-07-112-16/+23
| | | | .. instead of InputStream::LockRead(). The former is cancellable.
* decoder/sndfile: log seek errorsMax Kellermann2014-07-111-1/+4
|
* decoder/audiofile: log seek errorsMax Kellermann2014-07-111-0/+1
|
* decoder/audiofile: fix typo in commentMax Kellermann2014-07-111-1/+1
|
* PlaylistEdit: postpone UpdateQueuedSong() when adding multiple songsMax Kellermann2014-07-119-1/+111
| | | | | | | | | | Implement a "bulk" edit mode that postpones both UpdateQueuedSong() and OnModified(). This way, the playlist version gets incremented only once. More importantly: when adding multiple songs to a queue that consists of only one song, the first song that got added will always be played next. By postponing this choice, all newly added songs get a chance to become the next song. Fixes the second (and last) part of Mantis ticket 0004005.
* QueueCommands: make "result" more localMax Kellermann2014-07-111-3/+2
|
* Playlist: randomize next song when enabling "random" mode while not playingMax Kellermann2014-07-112-1/+4
| | | | | | Don't restore the current song after shufflung when MPD is stopped (but still remembers the current song internally). Fixes the first part of Mantis ticket 0004005.
* PlaylistControl: update code commentMax Kellermann2014-07-111-1/+1
|
* decoder/dsd: use decoder_read_full() where appropriateJoff2014-07-095-31/+20
| | | | | | | Addresses Mantis ticket 0004015. [mk: use decoder_read_full() only when needed, and a few formal changes]
* decoder/audiofile: use decoder_read_full()Max Kellermann2014-07-092-1/+7
| | | | | Works around WAV stream playback bug, because libaudiofile does not like partial reads (Mantis 0004028).
* DecoderAPI: add function decoder_read_full()Max Kellermann2014-07-095-12/+67
| | | | Move code from the "mad" plugin.
* DecoderAPI: add function decoder_skip()Max Kellermann2014-07-095-14/+57
| | | | Move code from the "mad" plugin.
* test: merge duplicate code to FakeDecoderAPI.cxxMax Kellermann2014-07-094-157/+113
|
* decoder/audiofile: use decoder_read()Max Kellermann2014-07-092-15/+26
| | | | | .. instead of InputStream::LockRead(). The former is cancellable.
* playlist/PlsPlaylistPlugin: fix build failure due to missing stdio.h includeGustavo Zacarias2014-07-091-0/+1
| | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
* output/HttpdClient: fix build failure due to missing stdio.h includeGustavo Zacarias2014-07-091-0/+1
| | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
* decoder/OpusDecoderPlugin: fix build failure due to missing stdio.h includeGustavo Zacarias2014-07-091-0/+1
| | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
* decoder/AudiofileDecoderPlugin: fix build failure due to missing stdio.h includeGustavo Zacarias2014-07-091-0/+1
| | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
* db/proxy: fall back to recursive walk on old libmpdclient/MPDMax Kellermann2014-06-236-1/+64
| | | | | Error message was 'too few arguments for "find"' because the "base" constraint was not supported, and no other constraints remained.
* db/proxy: use mpd_song_get_{start,end}() only with libmpdclient >= 2.3Max Kellermann2014-06-232-0/+7
|
* configure.ac: prepare for 0.18.12Max Kellermann2014-06-232-1/+3
|
* release v0.18.11Max Kellermann2014-05-121-1/+1
|
* decoder/OggUtil: allow skipping up to 32 kB after seekMax Kellermann2014-04-292-1/+3
| | | | | | | Fixes missing song length on high-latency Opus files. According to tests with 320 kbit/s opus files with 60ms packets, we need to skip up to 29 kB.
* DeferredMonitor: fix race condition when using GLib event loopMax Kellermann2014-04-263-11/+23
| | | | | | | | | Turns out the lock-free code using atomics was not thread-safe. The given callback could be invoked by GLib before the source_id attribute was assigned. This commit changes the DeferredMonitor class to use a Mutex to block the event loop until source_id is assigned. This bug does not exist in the 0.19 branch because it does not use the GLib main loop anymore.