aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* 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
|
* Compiler.h: add macro GCC_OLDER_THAN()Max Kellermann2014-12-262-4/+12
|
* util/Manual: reimplement GCC_CHECK_VERSION() using GCC_MAKE_VERSION()Max Kellermann2014-12-261-4/+3
|
* Compiler.h: add macro GCC_MAKE_VERSION()Max Kellermann2014-12-261-6/+4
|
* playlist/embcue: fix filename suffix detectionk442014-12-161-1/+1
| | | | | The definition of the playlist_plugin struct member of the embcue plugin was incorrect.
* decoder/ffmpeg: detect and fix negative time stampsMax Kellermann2014-12-151-4/+7
| | | | | Works around assertion failure due to something that appears to be a (minor) FFmpeg bug.
* decoder/ffmpeg: support FFmpeg 2.5Max Kellermann2014-12-081-0/+5
| | | | | | Version 2.5 fixed an API oddity, however it broke API compatibility, at least with C++. Disable the workaround when a libavformat version is detected that is recent enough.
* thread/Id: drop "::" prefix before pthread function namesMax Kellermann2014-12-081-2/+2
| | | | | | The "::" to explicitly refer to the global namespace appeared like a good idea in C++, but it breaks with C libraries that implement standard functions using macros (e.g. musl).
* archive/zzip: fix inverted error handlerMax Kellermann2014-11-241-4/+5
| | | | | Set the Error when zzip_seek()==-1 and not on success. Fixes a crash after seeking.
* Client: assume uid==0 is local socketMax Kellermann2014-11-182-2/+2
| | | | | A negative uid value means it's not a "local socket" (PF_LOCAL). uid==0 means user "root" connected.
* event/ServerSocket: fix get_remote_uid() error valueMax Kellermann2014-11-181-1/+1
| | | | Must return -1 on error, not 0. 0 is root.
* Construct a Null AllocatedPath if the filename conversion into UTF8 failedFlorent Le Coz2014-11-111-1/+5
|
* decoder/ffmpeg: support opusMax Kellermann2014-11-101-1/+2
|
* input/curl: fix curl_easy_setopt() parameter typesMax Kellermann2014-11-021-4/+4
|
* Decoder, Playlist: ignore URI query string for plugin detectionMax Kellermann2014-11-022-6/+8
| | | | Use the new uri_get_suffix() overload that removes the query string.
* util/UriUtil: add uri_get_suffix() overload that ignores query stringMax Kellermann2014-11-022-0/+28
|
* PlaylistFile: don't allow empty playlist nameMax Kellermann2014-11-021-0/+4
|
* playlist/m3u: recognize the file suffix ".m3u8"Max Kellermann2014-11-022-1/+3
|
* decoder/faad: remove workaround for ancient libfaad2 ABI bugMax Kellermann2014-11-021-10/+2
| | | | | | | | | | | | | | | | | | | | | | | | Many years ago, FAAD had a serious ABI bug: the NeAACDecInit() prototype in its header declared the "samplerate" parameter to be "unsigned long *", but internally, the function assumed it was "uint32_t *" instead. On 32 bit machines, that was no difference, but on 64 bit, this left one portion of the return value uninitialized; and worse, on big-endian, the wrong word was filled. This bug had to be worked around in MPD (commit 9c4e97a6). A few months later, the bug was fixed in the FAAD CVS in commit 1.117 on file libfaad/decoder.c; the commit message was: "Use public headers internally to prevent duplicate declarations" The commit message was too brief at best; the problem was not duplicate declarations, but a prototype mismatch. No mention of the bug fix in the ChangeLog. The MPD project never learned about this bug fix, and so MPD would always pass a "uin32_t *" dressed up as a "unsigned long *". Nearly 6 years later, it's about time to fix this second ABI problem. Let's kill the workaround!
* decoder/ffmpeg: recognize MIME type audio/aacpSteven OBrien2014-11-021-0/+1
|
* output/roar: remove unnecessary "volatile" keywordMax Kellermann2014-11-021-1/+1
| | | | | A mutex acts as a memory barrier, and thus "volatile" is not necessary.
* TagString: use g_strndup() for unterminated stringMax Kellermann2014-11-021-1/+1
| | | | Fixes buffer overflow bug.
* CommandLine: update copyright yearMax Kellermann2014-09-261-2/+2
|
* configure.ac: add option to disable the DSD decodersMax Kellermann2014-09-261-0/+2
| | | | Allow building a smaller MPD binary for people who don't need DSD.
* playlist/embcue: change name string to "embcue"Max Kellermann2014-09-261-1/+1
| | | | The name "cue" was listed twice in "mpd --version".
* OutputThread: retain negative mix ratioMax Kellermann2014-09-181-1/+10
| | | | Fixes MixRamp breakage.
* command/list: reset used size after the list has been processedAndrzej Rybczak2014-09-181-0/+1
|
* thread/Posix{Cond,Mutex}: don't ues PTHREAD_*_INITIALIZER on NetBSDMax Kellermann2014-09-133-1/+29
| | | | | | | | | | On NetBSD, PTHREAD_MUTEX_INITIALIZER and PTHREAD_COND_INITIALIZER are not compatible with C++11 "constexpr" (see Mantis ticket 0004110). As a workaround, don't ues "constexpr", and use the functions pthread_mutex_init(), pthread_mutex_destroy(), pthread_cond_init() and pthread_cond_destroy() instead. This adds some runtime overhead, but is portable to POSIX implementations that have awkward initializer macros.
* decoder/ffmpeg: use memset() to initialize AVProbeDataMax Kellermann2014-09-091-0/+6
|
* decoder/ffmpeg: pass MIME type to ffmpeg/libav version 11Max Kellermann2014-09-071-0/+7
| | | | | | | That attribute was uninitialized before, which could crash libavformat. See Debian bug 760669
* Decoder*: add more assertionsMax Kellermann2014-09-072-0/+7
|
* DecoderThread: clear the pipe when handling late SEEKMax Kellermann2014-09-071-1/+11
| | | | | See code comment. Fixes assertion failure in decoder_command_finished().
* decoder/audiofile: fix crash after seekingMax Kellermann2014-09-061-1/+1
| | | | | | Log call was added to the wrong branch. Fixes regression by commit ca1a1149
* protocol/ArgParser: fix integer overflow in parse_range()Max Kellermann2014-09-041-2/+2
| | | | | | | | | | | | | Casting std::numeric_limits<unsigned>::max() to "long" leads to an overflow if sizeof(unsigned)==sizeof(long), and the result will be -1. This happens on some 32 bit architectures, for example ARM and WIN32. Workaround: use std::numeric_limits<int>::max(), which is the largest signed integer. Since sizeof(long)>=sizeof(int), this will never overflow. Fixes Mantis ticket 0004080.
* PlaylistControl: use SeekSongOrder(current) to keep current songMax Kellermann2014-08-311-1/+1
| | | | | | The "current" attribute is a "song order", not a "song position". This is usually the same - except in random mode. Fixes Mantis ticket 0004073.
* PlaylistControl: move code to new method SeekSongOrder()Max Kellermann2014-08-312-7/+19
|
* output/alsa: fix endless loop at end of file in dsd_usb modeMax Kellermann2014-08-311-0/+9
|
* output/{alsa,oss}: add assertionsMax Kellermann2014-08-312-0/+6
|
* decoder/gme: fix song durationMax Kellermann2014-08-291-1/+1
| | | | The unit of gme_info_t::length is milliseconds, not centiseconds.
* event/TimeoutMonitor: really reset "active" flag before invoking OnTimeout()Max Kellermann2014-08-241-1/+1
| | | | The previous commit was broken. D'oh!
* event/TimeoutMonitor: reset "active" flag before invoking OnTimeout()Max Kellermann2014-08-241-1/+3
| | | | | | The IsActive() method returned true even if the timer was not active, after it completed once. This broke the state file timer, and the state file was not saved periodically.
* system/ByteOrder: <endian.h> is a non-standard header that only Linux provides.Thomas Klausner2014-08-231-0/+10
|
* decoer/dsdiff: fix endless loop on malformed fileMax Kellermann2014-08-211-1/+1
| | | | Same bug as in the previous commit.