aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
* DecoderAPI: log without holding mutexMax Kellermann2013-11-081-2/+2
|
* PlayerThread: fix stuck MPD after song change (0.18.2 regression)Max Kellermann2013-11-081-4/+8
| | | | | | | | | | | | | Commit 77c63511 caused MPD to become stuck right after a song change. The problem was that at some point, the MusicBuffer became full, and the DecoderThread working on the next song waits for the PlayerThread. However, the PlayerThread was stuck in a loop of g_usleep() calls, and never bothered to tell the DecoderThread that the MusicBuffer is not full anymore. This bug is very old, but its chance to occur went from nearly 0% to nearly 100%. The fix is to wake up the DecoderThread before waiting for it. As a side effect, I replaced the g_usleep() call with a Cond::Wait() call.
* DecoderInternal: simplify need_chunks()Max Kellermann2013-11-081-11/+4
| | | | | Remove the "do_wait" parameter which is always true. Check only command==NONE and merge the "return" statements.
* DecoderInternal: don't wake up player in need_chunksMax Kellermann2013-11-081-1/+0
| | | | Nothing of interest happens here. No need to wake up the player.
* input/cdio_paranoia: add setting "default_byte_order"Max Kellermann2013-11-071-2/+26
| | | | | | Allows big-endian users to configure the fallback byte order to little-endian. Without this setting, MPD assumes native byte order if the CD drive can't decide.
* AvahiPoll: don't close the dbus socketMax Kellermann2013-11-071-0/+4
| | | | Avoid closing it twice.
* input/curl: unregister removed sockets from epollMax Kellermann2013-11-071-1/+9
| | | | Fixes a crash bug. See code comment.
* OutputControl: reduce the number of OutputThread wakeupsMax Kellermann2013-11-064-2/+30
| | | | | Wake up the OutputThread only if it hasn't already been woken up and if it isn't already in the playback loop.
* PlayerThread: reduce the number of DecoderThread wakeupsMax Kellermann2013-11-061-2/+14
| | | | | | | After the number of decoded chunks has fallen below the threshold, the PlayerThread woke up the DecoderThread over and over. This commit adds a boolean flag that avoids these duplicate wakeups, and thus reduces the number of system calls.
* DecoderControl: reduce the number of PlayerThread wakeupsMax Kellermann2013-11-063-4/+26
| | | | | Wake up the PlayerThread only if it is really waiting for the decoder. This greatly reduces the number of system calls in the DecoderThread.
* DecoderInternal: lock DecoderControl in _flush_chunk()Max Kellermann2013-11-062-0/+4
| | | | Must hold the Mutex to signal the Cond object safely.
* DecoderInternal: wake up the player thread in _flush_chunk()Max Kellermann2013-11-062-4/+2
| | | | Merge duplicate code.
* event/FullyBufferedSocket: remove obsolete TODO commentMax Kellermann2013-11-061-16/+0
| | | | We have a better solution now.
* ClientRead: "close" flushes the output bufferMax Kellermann2013-11-063-1/+12
| | | | | | Add a new CommandResult code called "FINISH" which, unlike "CLOSE", will attempt to flush the output buffer. This is a one-shot attempt; it will do one write, and not try again.
* event/FullyBufferedSocket: try to write without extra roundtripMax Kellermann2013-11-062-17/+33
| | | | | | Postpone the write using IdleMonitor instead of scheduling a write event. This reduces the number of system calls, because we don't need to register and unregister the write event in epoll.
* event/FullyBufferedSocket: make WriteFromBuffer() protectedMax Kellermann2013-11-062-4/+4
| | | | .. and rename it to Flush().
* PlayerThread: enable buffering when starting playbackMax Kellermann2013-11-061-1/+1
| | | | | | For some reason, this got lost in commit 975deca8. Re-enabling this fixes stuttering at the beginning of radio streams.
* input/curl: fix bug with redirected streamsMax Kellermann2013-11-061-110/+138
| | | | | | | | | | | Migrate from the old curl_multi_perform() API to the newer curl_multi_socket_action() API (since CURL 7.16). This allows working around a bug with HTTP redirections with epoll: when CURL closes a socket and the new one happens to have the same file number, MPD did not have a chance to remove the old one from epoll and subsequently attempted to use EPOLL_CTL_MOD, which was not allowed by epoll, because it's a new socket now.
* input/curl: move code into class CurlMultiMax Kellermann2013-11-061-43/+66
| | | | | Move all global variables there, and keep just one global variable: the pointer to the CurlMulti instance. Prepares for the next commit.
* input/curl: remove the global list of requestsMax Kellermann2013-11-061-13/+0
| | | | Unused.
* input/curl: don't abort all requests on curl_multi_perform() errorMax Kellermann2013-11-061-40/+7
| | | | Eliminate some complicated code that's not worth the trouble.
* input/curl: use CURLOPT_PRIVATEMax Kellermann2013-11-061-5/+7
| | | | Replaces the loop in input_curl_find_request().
* event/SocketMonitor: add method Abandon()Max Kellermann2013-11-064-2/+32
|
* system/ByteOrder: fix big-endian supportMax Kellermann2013-11-051-2/+2
| | | | D'oh!
* playlist/pls: fix reversed song orderMax Kellermann2013-11-051-1/+0
| | | | | Remove the forward_list::reverse() call. It was not necessary, because pls_parser() already reads the playlist in reverse order.
* decoder/dsf: enable DSD128Jurgen Kramer2013-11-051-2/+2
|
* add various missing headersNatanael Copa2013-11-053-0/+3
| | | | | | Fixes building with gcc-4.8 Signed-off-by: Natanael Copa <ncopa@alpinelinux.org>
* output/alsa: avoid endless loop in Raspberry Pi workaroundMax Kellermann2013-11-041-1/+17
| | | | See code comment.
* event/ServerSocket: don't abort if IPv6 is not availableMax Kellermann2013-11-041-1/+19
| | | | First check if an IPv6 socket can be created.
* Log: add level "DEFAULT"Max Kellermann2013-11-0414-45/+72
| | | | | | Map LogLevel::INFO to G_LOG_LEVEL_INFO, and LogLevel::DEFAULT to G_LOG_LEVEL_MESSAGE. Now client connect/disconnect message are only logged on log_level "secure".
* Log: document the LogLevel itemsMax Kellermann2013-11-041-0/+16
|
* event/ServerSocket: fix assertion failureMax Kellermann2013-11-041-1/+1
| | | | Regression from previous commit. D'oh!
* event/ServerSocket: open sockets in the order they were configuredMax Kellermann2013-11-042-5/+5
| | | | Use a std::list which can be appended at the end.
* system/resolver: return path of UNIX domain socketsMax Kellermann2013-11-041-7/+15
| | | | getnameinfo() doesn't work well - it always returns "localhost".
* ClientRead: always ignore whitespace at the end of the lineMax Kellermann2013-11-041-2/+7
|
* filter/AutoConvert: modify child_audio_format, not in_audio_formatMax Kellermann2013-11-041-2/+2
| | | | | | | | This prevented using the "volume_normalization" feature with some codecs (e.g. mp3), because the normalization code requires 16 bit samples. If the codec happens to deliver formats other than S16, the AutoConvert filter succeeds to initialize the conversion filter, but the returned input audio format was wrong.
* CommandLine: simplified program name lineMax Kellermann2013-10-311-1/+1
|
* system/FatalError: use _exit() instead of exit()Max Kellermann2013-10-301-2/+2
| | | | | Skip the global destructors. We don't need them here - we bail out as quickly as we can.
* system/FatalError: move code to Abort()Max Kellermann2013-10-301-3/+10
|
* *: update copyright year to 2013Max Kellermann2013-10-30116-116/+116
|
* ClientList: disconnect all clients in destructorMax Kellermann2013-10-301-0/+3
| | | | Fixes assertion failure.
* mixer/software: fix double free bugMax Kellermann2013-10-301-3/+15
|
* PlaylistState: ignore "mixrampdelay:nan"Max Kellermann2013-10-301-1/+7
| | | | mixramp_delay==nan() causes severe problems with cross-fading.
* increase default buffer size to 4 MBMax Kellermann2013-10-301-1/+1
| | | | | | 2 MB was too small for cross-fading a 24 bit file. Increasing to 4 MB is still not too large for weak machines, but is enough for cross-fading.
* db/proxy: remove obsolete #undefsMax Kellermann2013-10-301-2/+0
|
* db/proxy: auto-reconnectMax Kellermann2013-10-301-14/+64
|
* ReplayGainInfo: use large negative value instead of infinityMax Kellermann2013-10-302-4/+4
| | | | The last piece to allow -ffast-math.
* PlayerCommands: show CrossFade/MixRamp only if enabledMax Kellermann2013-10-301-4/+10
|
* PcmMix: use negative value instead of NaN for additionMax Kellermann2013-10-303-7/+6
| | | | Avoid NaN to allow -ffast-math.
* CrossFade: eliminate NaN from mixramp_interpolate()Max Kellermann2013-10-301-4/+4
| | | | Use a boolean flag instead.
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-14 06:04:42 +0000