| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Use stat() instead of g_file_test() to detect other types of errors,
such as "permission denied".
|
| |
|
|
|
|
| |
Convert to padded 24 bit samples, instead of falling back to 16 bit.
|
|
|
|
|
|
| |
This fixes a buffer corruption bug; pcm_buffer is not designed to be a
persistent buffers, and will discard anything between two consecutive
calls.
|
|
|
|
|
|
| |
This fixes a buffer corruption bug; pcm_buffer is not designed to be a
persistent buffers, and will discard anything between two consecutive
calls.
|
|
|
|
|
|
| |
This fixes a buffer corruption bug; pcm_buffer is not designed to be a
persistent buffers, and will discard anything between two consecutive
calls.
|
|
|
|
| |
Fixes assertion failure.
|
|
|
|
|
|
|
| |
WinAPI explicitly declares filesystem encoding.
It can be determined by GetACP().
Use that instead of Glib routine that always "detects" UTF-8 on Win32,
which is incorrect for MPD case.
|
|
|
|
|
|
| |
Ensure that WINVER is defined early enough, so other system headers
won't fall back to their default value. Specifically, this solves a
build failure (-Werror) with mingw-w64 ("WINVER redefined").
|
|
|
|
|
| |
The event pipe is not a socket, and the patch that introduced
g_io_channel_new_socket() to the event pipe library was wrong.
|
| |
|
| |
|
|
|
|
| |
.. instead of failing playback completely.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Right now, a playlist with absolute pathnames can only add songs that
are in the same the directory of the playlist or under it.
If uri is an absolute pathname and base_uri is set,
playlist_check_translate_song() will check that base_uri is a prefix
of uri, excluding every other song in the music directory outside
base_uri.
I think in this case base_uri should be completely ignored (and made
NULL) and uri should just be checked against music root directory.
|
|
|
|
|
|
|
| |
Previously, the condition "defined(play_audio_format)" was used to see
if an output device has been opened, but if the device had failed on
startup, an assertion failure could occur. This patch adds a separate
flag.
|
|
|
|
|
| |
Now that the player thread can handle SEEK commands while not (yet)
playing, we can remove the "pc.state" check from pc_seek().
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When playing a CUE track, the player thread waited for the decoder to
become ready, and then sent a SEEK command to the beginning of the CUE
track. If that is near the start of the song file, and the track is
short enough, the decoder could have finished decoding already at that
point, and seeking fails.
This commit makes this initial seek more robust: instead of letting
the player thread deal with the difficult timings, let the decoder API
emulate a SEEK command, and return it to the decoder plugin, as soon
as the plugin finishes its initialization.
|
|
|
|
| |
Wrap close(), use closesocket() on WIN32/WinSock.
|
|
|
|
|
| |
Allow enabling the plugin explicitly without running Solaris, to test
the build.
|
|
|
|
| |
Wrong variable name.
|
|
|
|
| |
Add -DNDEBUG to AM_CPPFLAGS.
|
|
|
|
| |
Use flag AV_TIME_BASE.
|
| |
|
|
|
|
| |
Reduce heap usage by reducing the number of malloc() / free() calls.
|
|
|
|
|
|
| |
D'oh, we were reading 16 bit integers instead of 32 bit integers!
That caused silence when trying to play a 32 bit input file on a 24
bit sound card (e.g. USB sound chips with 24 bit packed samples).
|
| |
|
| |
|
|
|
|
| |
Eliminate an unnecessary source of deadlocks.
|
|
|
|
|
| |
The pkg-config file was added by the Debian package maintainers, and
unfortunately, the rest of the world doesn't have it.
|
|
|
|
|
|
| |
Don't abort the configure script when avahi could not be
auto-detected. It previously did, because there was no custom "fail"
action for PKG_CHECK_MODULES.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The output thread could hang indefinitely after finishing CANCEL,
because it could have missed the signal while the output was not
unlocked in ao_command_finished().
This patch removes the wait() call after CANCEL, and adds the flag
"allow_play" instead. While this flag is set, playback is skipped.
With this flag, there will not be any excess wait() call after the
pipe has been cleared.
This patch fixes a bug that causes mpd to discontinue playback after
seeking, due to the race condition described above.
|
|
|
|
|
| |
Be sure to stop the operation at some point when the server isn't
responding.
|
|
|
|
|
| |
Check if the stream is suspended; wake up the main loop when it
becomes suspended.
|
|
|
|
| |
Unlock the mainloop in all code paths.
|
| |
|
|
|
|
|
|
| |
This warning should only be logged when we really received something.
When the client disconnects, G_IO_IN is triggered, and the read
returns G_IO_STATUS_EOF.
|
|
|
|
|
|
|
| |
In the "vorbis" plugin, this is a copy of the old flush() method,
while flush() gets a lot of code remove, it just sets the "flush" flag
and nothing else. It doesn't start a new stream now, which should fix
a few problems in some players.
|
| |
|
|
|
|
| |
It used to ignore the decoder_data() return value.
|
|
|
|
|
|
|
| |
paused
Use a shortcut in playlist_seek_song(), don't call
playlist_play_order() because that would reset the "paused" state.
|
| |
|
| |
|
|
|
|
|
| |
Don't calculate the song duration when the sample rate is 0 (division
by zero crash).
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Keep those when scanning for empty directories. The check in
playlist_vector_is_empty() was missing.
|
|
|
|
|
| |
g_path_get_dirname() returns "." when there is no directory name in
the given path. This patch adds a workaround for that.
|