aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* decoer/dsf: fix endless loop on malformed fileMax Kellermann2014-08-211-1/+1
| | | | | | | 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