Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | decoder/faad: use adts_check_frame() in faad_song_duration() | Max Kellermann | 2014-07-12 | 1 | -2/+1 | |
| | | | | Eliminate more duplicate code. | |||||
* | decoder/faad: test "seekable" after ADTS frame check | Max Kellermann | 2014-07-12 | 1 | -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 Kellermann | 2014-07-12 | 1 | -14/+17 | |
| | | | | Merge some duplicate code. | |||||
* | decoder/faad: remove unnecessary read | Max Kellermann | 2014-07-12 | 1 | -2/+0 | |
| | | | | Eliminate some overhead when the caller doesn't need the buffer. | |||||
* | DecoderBuffer: add "pure" attributes | Max Kellermann | 2014-07-12 | 1 | -0/+4 | |
| | ||||||
* | DecoderBuffer: add method _clear() | Max Kellermann | 2014-07-12 | 3 | -8/+12 | |
| | ||||||
* | decoder/faad: check sample_rate, not frames_per_second | Max Kellermann | 2014-07-11 | 1 | -2/+4 | |
| | | | | Checking the integer is faster, easier and more reliable. | |||||
* | decoder/faad: make variables more local | Max Kellermann | 2014-07-11 | 1 | -71/+49 | |
| | ||||||
* | decoder/faad: use MAX_CHANNELS | Max Kellermann | 2014-07-11 | 1 | -4/+2 | |
| | | | | .. instead of declaring a new constant. | |||||
* | decoder/sndfile: use decoder_read_full() | Max Kellermann | 2014-07-11 | 1 | -16/+6 | |
| | | | | | Replaces the loop in sndfile_vio_read(), eliminating duplicate and fragile code. | |||||
* | decoder/sndfile: use decoder_read() | Max Kellermann | 2014-07-11 | 1 | -16/+22 | |
| | | | | .. instead of InputStream::LockRead(). The former is cancellable. | |||||
* | decoder/sndfile: log seek errors | Max Kellermann | 2014-07-11 | 1 | -1/+4 | |
| | ||||||
* | decoder/audiofile: log seek errors | Max Kellermann | 2014-07-11 | 1 | -0/+1 | |
| | ||||||
* | decoder/audiofile: fix typo in comment | Max Kellermann | 2014-07-11 | 1 | -1/+1 | |
| | ||||||
* | PlaylistEdit: postpone UpdateQueuedSong() when adding multiple songs | Max Kellermann | 2014-07-11 | 7 | -1/+109 | |
| | | | | | | | | | | 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 local | Max Kellermann | 2014-07-11 | 1 | -3/+2 | |
| | ||||||
* | Playlist: randomize next song when enabling "random" mode while not playing | Max Kellermann | 2014-07-11 | 1 | -1/+3 | |
| | | | | | | 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 comment | Max Kellermann | 2014-07-11 | 1 | -1/+1 | |
| | ||||||
* | decoder/dsd: use decoder_read_full() where appropriate | Joff | 2014-07-09 | 4 | -31/+19 | |
| | | | | | | | Addresses Mantis ticket 0004015. [mk: use decoder_read_full() only when needed, and a few formal changes] | |||||
* | decoder/audiofile: use decoder_read_full() | Max Kellermann | 2014-07-09 | 1 | -1/+6 | |
| | | | | | Works around WAV stream playback bug, because libaudiofile does not like partial reads (Mantis 0004028). | |||||
* | DecoderAPI: add function decoder_read_full() | Max Kellermann | 2014-07-09 | 3 | -12/+31 | |
| | | | | Move code from the "mad" plugin. | |||||
* | DecoderAPI: add function decoder_skip() | Max Kellermann | 2014-07-09 | 3 | -14/+25 | |
| | | | | Move code from the "mad" plugin. | |||||
* | decoder/audiofile: use decoder_read() | Max Kellermann | 2014-07-09 | 1 | -15/+24 | |
| | | | | | .. instead of InputStream::LockRead(). The former is cancellable. | |||||
* | playlist/PlsPlaylistPlugin: fix build failure due to missing stdio.h include | Gustavo Zacarias | 2014-07-09 | 1 | -0/+1 | |
| | | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> | |||||
* | output/HttpdClient: fix build failure due to missing stdio.h include | Gustavo Zacarias | 2014-07-09 | 1 | -0/+1 | |
| | | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> | |||||
* | decoder/OpusDecoderPlugin: fix build failure due to missing stdio.h include | Gustavo Zacarias | 2014-07-09 | 1 | -0/+1 | |
| | | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> | |||||
* | decoder/AudiofileDecoderPlugin: fix build failure due to missing stdio.h include | Gustavo Zacarias | 2014-07-09 | 1 | -0/+1 | |
| | | | | Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> | |||||
* | db/proxy: fall back to recursive walk on old libmpdclient/MPD | Max Kellermann | 2014-06-23 | 5 | -1/+63 | |
| | | | | | 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.3 | Max Kellermann | 2014-06-23 | 1 | -0/+5 | |
| | ||||||
* | decoder/OggUtil: allow skipping up to 32 kB after seek | Max Kellermann | 2014-04-29 | 1 | -1/+1 | |
| | | | | | | | 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 loop | Max Kellermann | 2014-04-26 | 2 | -11/+22 | |
| | | | | | | | | | 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. | |||||
* | decoder/sndfile: work around libsndfile bug on partial read | Marcello Desantis | 2014-04-09 | 1 | -6/+20 | |
| | ||||||
* | PlaylistEdit: don't interrupt playback when current song gets deleted | Weng Xuetian | 2014-04-09 | 1 | -10/+8 | |
| | ||||||
* | decoder/ffmpeg: handle unknown stream start time | Max Kellermann | 2014-03-18 | 1 | -2/+25 | |
| | ||||||
* | decoder/ffmpeg: pass AVSEEK_FLAG_ANY to av_seek_frame() | Max Kellermann | 2014-03-18 | 1 | -1/+1 | |
| | | | | | | This corrects a major mistake from commit 724a59aa - there was one small thing that commit was supposed to do, and it failed. AV_TIME_BASE is not a seek flag. | |||||
* | decoder/gme: fix memory leak in container_scan() | Max Kellermann | 2014-03-06 | 1 | -0/+1 | |
| | ||||||
* | output/alsa: remove the obsolete Raspberry Pi workaround | Max Kellermann | 2014-03-02 | 1 | -36/+0 | |
| | | | | Has been superseded by the previous commit. | |||||
* | output/alsa: call snd_pcm_prepare() after snd_pcm_drop() | Max Kellermann | 2014-03-02 | 1 | -0/+25 | |
| | | | | | | | Don't wait for an optimistic write to fail. This is an improved workaround for the infamous Raspberry Pi bug (see commit af991765). It works much better and comes without the negative side effects. The old workaround is now obsolete. | |||||
* | event/SignalMonitor: fix build failure due to missing signal.h include | Max Kellermann | 2014-03-02 | 1 | -0/+2 | |
| | ||||||
* | AllCommands: "findadd" requires the "add" permission | Max Kellermann | 2014-02-27 | 1 | -1/+1 | |
| | ||||||
* | output/pulse: remove bogus g_free() call | Max Kellermann | 2014-02-24 | 1 | -2/+0 | |
| | ||||||
* | event/SignalMonitor: unblock signals after fork | Max Kellermann | 2014-02-18 | 1 | -1/+22 | |
| | | | | Fixes hanging child process in the "pipe" output plugin. | |||||
* | IcyMetadataParser: more robust tag parser | Max Kellermann | 2014-01-27 | 1 | -21/+71 | |
| | | | | | Allow semicolons and single quotes in the stream title. This is not part of any specification, but found in real life. | |||||
* | Queue: rename struct queue to Queue | Max Kellermann | 2014-01-20 | 10 | -53/+53 | |
| | | | | | Works around a build failure on Solaris because annoyingly, Solaris reserves the name "queue". This rename was pending anyway. | |||||
* | Queue: make the constructor "explicit" | Max Kellermann | 2014-01-20 | 1 | -1/+1 | |
| | ||||||
* | decoder/ffmpeg: support libav v10_alpha1 | Max Kellermann | 2014-01-15 | 1 | -1/+11 | |
| | ||||||
* | decoder/ffmpeg: include cleanup | Max Kellermann | 2014-01-15 | 2 | -4/+1 | |
| | ||||||
* | decoder/ffmpeg: check for av_samples_get_buffer_size() errors | Max Kellermann | 2014-01-15 | 1 | -0/+3 | |
| | | | | Fixes potential nullptr dereference. | |||||
* | util/PeakBuffer: fix nullptr dereference when peak_size==0 | Max Kellermann | 2014-01-15 | 1 | -2/+3 | |
| | ||||||
* | SongFilter, TagConfig: cast TAG_NUM_OF_ITEM_TYPES to integer | Max Kellermann | 2014-01-15 | 2 | -2/+2 | |
| | | | | Fixes clang warning. |