aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* gme: don't loop forever, fall back to GME's default play lengthMax Kellermann2015-10-261-3/+3
| | | | Fixes http://bugs.musicpd.org/view.php?id=4432
* decoder/gme: free the gme_info_t as early as possibleMax Kellermann2015-10-261-6/+7
|
* decoder/gme: call decoder_seek_error() on seek errorMax Kellermann2015-10-261-2/+4
|
* output/Command: flush the mixer cache when enabling/disabling outputMax Kellermann2015-10-261-0/+8
| | | | Fixes mixer lag (http://bugs.musicpd.org/view.php?id=4425).
* decoder/ffmpeg: fix crash due to wrong avio_alloc_context() callMax Kellermann2015-10-261-4/+13
| | | | | | | | Allocate the buffer dynamically using av_malloc(), and free AVIOContext.buffer in the destructor, as mandated by the libavformat documentation. Fixes http://bugs.musicpd.org/view.php?id=4446
* decoder/mpcdec: use SampleTraits<SampleFormat::S24_P32>Max Kellermann2015-10-161-11/+13
| | | | | | Eliminates some duplicate code, and as a side effect, this works around clang 3.8 compiler warning because a negative value was shifted.
* decoder/mpcdec: use Clamp()Max Kellermann2015-10-161-6/+2
|
* unix/Daemon, playlist/...: remove unused Domain variablesMax Kellermann2015-10-167-21/+0
|
* encoder/flac: fix crash with 32 bit playbackMax Kellermann2015-10-161-2/+2
| | | | | | | Copy to encoder->audio_format *after* adjusting the sample format to S24_P32. Fixes http://bugs.musicpd.org/view.php?id=4433
* tag/ApeLoader: fix buffer overflow after unterminated keyMax Kellermann2015-10-161-6/+6
|
* PlayerThread: start the decoder on PlayerCommand::QUEUEMax Kellermann2015-06-211-0/+6
| | | | Fixes missing SongBorder() call, which causes "single" mode breakage.
* DecoderAPI: discard unused song tag earlyMax Kellermann2015-06-211-2/+5
| | | | | If there's a stream tag, don't let the song tag override it in the next update_stream_tag() call.
* DecoderThread: set Decoder::song_tag only for local filesMax Kellermann2015-06-211-1/+5
| | | | | | | If the song tag comes from a stream, and MPD playback restarts, MPD would believe the tag should override the newly received tag. This makes the previous tag appear stuck. This change passes the song tag only if it's authoritative - i.e. if it's a song file.
* DecoderAPI: "move" the Tag objectMax Kellermann2015-06-201-1/+1
| | | | Reduce runtime overhead.
* OutputControl: fix fail_timer check right after bootingRomain Rollet2015-06-201-1/+2
| | | | | | Right after booting, the monotonic clock starts with a very small value, and AudioOutput::LockUpdate() may believe that the fail_timer has not recovered yet.
* system/PeriodClock: make IsDefined() "constexpr"Max Kellermann2015-06-201-1/+1
|
* decoder/ffmpeg: skip unwanted samples after seekingMax Kellermann2015-06-201-5/+48
| | | | | When seeking to the beginning of a packet, skip the samples that come before the desired time stamp.
* decoder/ffmpeg: use AVSEEK_FLAG_BACKWARD for seekingMax Kellermann2015-06-201-1/+5
| | | | | | Ask FFmpeg to seek to the next packet boundary *before* the seek position, so we don't miss audio data. Now we get too much, but we'll solve that in the next commit.
* decoder/ffmpeg: move code to StreamRelativePts()Max Kellermann2015-06-191-6/+20
|
* input/curl: trigger the condition variable in RequestDone()Max Kellermann2015-06-191-0/+2
| | | | Fixes deadlock on small responses.
* storage/nfs: fix deadlock when connectingThomas Guillem2015-05-291-2/+2
| | | | | | The Connect method can be called between Schedule and lock. In that case, when locked, the state is already set to CONNECTING of READY and the condition won't be signaled anymore.
* input/smbclient: fix DFF playbackjai2015-05-291-0/+1
|
* OpusEncoderPlugin: initialise granulepos so we end up with sane valuesMichal Nazarewicz2015-03-251-1/+1
| | | | | | | Not initialising granulepos leads to it having arbitrary values in the encoded stream including possibly negative values which are not valid and confuse opusdec. Explicitly initialise opus_encoder::granulepos to avoid that problem.
* fs/io/FileOutputStream: don't auto-delete file on WIN32Max Kellermann2015-02-061-0/+2
| | | | | | | | | The file handle is never reset to INVALID_HANDLE_VALUE, and thus the destructor will assume the operation shall be cancelled and will delete the temporary file. This was a major breakage for saving the database file and the state file.
* TagBuilder: allow adding duplicate tag types in Complement()Max Kellermann2015-02-061-1/+10
| | | | | | | | | | Build a table of pre-existing tag types before adding new items. The old way would check HasType() each time, which would return true after the first instance of that tag type had been added, preventing duplicate tag types to be merged. This broke duplicate tag types loaded from the state file, because this code path uses TagBuilder::Complement().
* db/update/Walk: use std::unique_ptr instead of std::auto_ptrMax Kellermann2015-01-291-1/+1
| | | | | std::auto_ptr is deprecated, and std::unique_ptr is much better anyway.
* Avoid integer overflow in MonotonicClock{S,MS,US}PHO2015-01-291-6/+6
| | | | | This is Darwin specific: the previous implementation was causing an integer overflow when base.numer is very large. On PPC Darwin, the timebase info is 1000000000/33330116 and this is too large for integer arithmetic.
* decoder/DsdLib: use new[] to allocate the ID3 bufferMax Kellermann2015-01-291-3/+5
| | | | | Don't abort the process if there's not enough memory. This buffer is not important and can be large.
* decoder/DsdLib: free ID3 buffer right after id3_tag_parse()Max Kellermann2015-01-291-5/+2
| | | | Merge two free() calls.
* decoder/DsdLib: raise ID3 tag limit to 1 MBMax Kellermann2015-01-291-1/+1
| | | | | A bug report was submitted with a 600 kB ID3 tag that could not be read by MPD.
* Merge branch 'v0.18.x' into v0.19.xMax Kellermann2015-01-2610-660/+3
|\
| * {playlist,input}/despotify: remove defunct pluginMax Kellermann2015-01-269-668/+0
| |
| * util/list: disable gcc5 warningMax Kellermann2015-01-231-0/+4
| | | | | | | | | | This file has been removed in newer MPD versions, so don't care about it now.
| * ClientProcess: cast enum to int before passing to printf()Max Kellermann2015-01-231-3/+3
| | | | | | | | Fixes gcc5 warning.
* | Test the existence of strndup(3) before using it.PHO2015-01-261-5/+6
| | | | | | | | This can eliminate the ad-hoc "#ifdef WIN32" and can also support other platforms lacking it as well (including Darwin 9).
* | thread/Name: include stdio.h if HAVE_PRCTLMax Kellermann2015-01-211-1/+4
| | | | | | | | | | Caused a build failure with uClibc because snprintf() was not available.
* | input/async: reset the "open" flag after seeking successfullyMax Kellermann2015-01-061-0/+5
| | | | | | | | | | | | | | | | | | Fixes a problem with the "curl" input plugin: IsEOF() always returns true because the "open" flag was cleared by CurlInputStream::RequestDone() when end-of-stream was reached. This flag stays false even when seeking to another position has succeeded. This patch resets the "open" flag to true after seeking successfully.
* | thread/Name: disable pthread_setname_np() on NetBSDMax Kellermann2015-01-061-2/+6
| | | | | | | | | | | | | | NetBSD's pthread_setname_np() prototype is incompatible with the rest of the world, and it requires to pass the string argument as a non-const pointer. Instead of working around this misdesign, I hereby disable the feature on NetBSD.
* | thread/Name: enable FormatThreadName() with prctl()Max Kellermann2015-01-061-1/+5
| | | | | | | | | | | | Add macro HAVE_THREAD_NAME which is set when any method to set the thread name is available. Use that macro in FormatThreadName() instead of just checking for HAVE_PTHREAD_SETNAME_NP.
* | thread/Name: indent preprocessor commandsMax Kellermann2015-01-061-3/+3
| |
* | db/simple: fix implicit nullptr/bool conversionMax Kellermann2014-12-263-5/+5
| | | | | | | | | | Return false on error, not nullptr.
* | fs/Traits, ...: work around -Wtautological-pointer-compareMax Kellermann2014-12-265-1/+32
| | | | | | | | New in clang 3.6.
* | db/lazy, input/mms: add "override" keywordsMax Kellermann2014-12-263-4/+6
| | | | | | | | Fixes -Winconsistent-missing-override (clang 3.6).
* | Merge branch 'v0.18.x' into v0.19.xMax Kellermann2014-12-2618-51/+110
|\|
| * db/proxy, output/shout: fix implicit nullptr/bool conversionMax Kellermann2014-12-262-4/+4
| | | | | | | | Return false on error, not nullptr.
| * db/{simple,proxy}, ...: add "override" keywordsMax Kellermann2014-12-267-22/+26
| | | | | | | | Fixes -Winconsistent-missing-override (clang 3.6).
| * util/{ASCII,UriUtil}, ...: work around -Wtautological-pointer-compareMax Kellermann2014-12-2610-0/+51
| | | | | | | | New in clang 3.6.
| * DatabaseGlue: convert nullptr check to assertionMax Kellermann2014-12-261-3/+2
| |
| * util/ASCII: fix indentMax Kellermann2014-12-261-10/+10
| |
| * Compiler.h: add macro CLANG_CHECK_VERSION()Max Kellermann2014-12-261-0/+7
| |