| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Thanks to clang for complaining.
|
|
|
|
| |
Otherwise OGGs can't be played.
|
|
|
|
|
| |
Without the ogg_stream_reset() call, the "e_o_s" flag never gets
reset, and libogg writes EOS packets over and over.
|
| |
|
| |
|
|
|
|
|
|
| |
With mono sound, jack_sample_size is smaller than frame_size (4 vs 2
bytes), and "space/jack_sample_size==0". That means mpd_jack_play()
will return 0, although no error has occurred.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Version 1.0.0 of the libao library added a new field to the
ao_sample_format struct. It is a char * named matrix. When
an ao_sample_format is allocated on the stack, this field contains
garbage. The proper course is to insure that is initialized to NULL.
NULL indicates that we do not want any mapping.
The struct is now initialized using a static initializer, and this
technique is compatible with all known versions of libao.
|
| |
|
|
|
|
|
|
|
| |
<stdbool.h> needs to be included unconditionally from definition of
NDEBUG, since »bool« doesn't get defined otherwise.
Signed-off-by: Andreas Wiese <aw-devel@meterriblecrew.net>
|
|
|
|
|
|
|
|
|
| |
When you don't explicitly set an output sample rate, liblame tries to
guess an output sample rate from the input sample rate. You would
think that this "guessing" consists of just setting both equal, but
that is not the case. For 44.1kHz at 96kbit/s, liblame chooses
32kHz. This patch explicitly configures the output sample rate, to
stop the bad guessing.
|
| |
|
|
|
|
| |
Fixes autotools warnings.
|
| |
|
|
|
|
| |
Cast the constant to dev_t, not to unsigned.
|
|
|
|
| |
Fixes the gcc warning "implicit declaration of function 'htons'".
|
| |
|
| |
|
|
|
|
|
|
| |
After popular demand, I've switched the order of "artist" and "title"
in the stream title. There is no standard, and there is no reliable
way to parse those from the stream title.
|
|
|
|
|
| |
The assertion added in MPD 0.15.14 was too much, it failed when the
MIME type of a stream was NULL.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
When one song is played twice, and the decoder is working on the
second "instance", but the first should be seeked, the check in
player_seek_decoder() may assume that it can reuse the decoder without
exchanging pipes. The last thing was the mistake: the pipe pointer
was different, which led to an assertion failure. This patch adds
another check which exchanges the player pipe.
|
|
|
|
| |
More abstraction for decoder_control.pipe.
|
|
|
|
| |
Some abstraction for decoder_control.pipe access.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Duplicate the "mime" attribute of the inner input_stream object,
instead of copying the pointer.
|
|
|
|
| |
Same as ".m4a".
|
|
|
|
|
|
| |
Change the assertion on "fail_timer==NULL" in OPEN to a runtime check.
This assertion crashed when the output thread failed while the player
thread was calling audio_output_open().
|
| |
|
|
|
|
| |
Protect the attributes "open" and "fail_timer".
|
|
|
|
| |
Protect the attributes "open" and "fail_timer".
|
| |
|
| |
|
|
|
|
|
|
| |
RFC 5334 10.3 defines the MIME type "audio/ogg". We could use
"application/ogg" as well, but we know for sure that we only emit
audio data.
|
| |
|
|
|
|
|
|
|
| |
The assumption that MIME type is set only once is not valid with CURL,
as URL redirections may update the MIME type.
This fixes bug #3044.
|
|
|
|
|
| |
This caused random data to be send via icy-server if the played
song had no tags.
|
|
|
|
|
|
|
| |
According to the CURL web site, curl_multi_timeout() was added in
version 7.15.4:
http://curl.haxx.se/libcurl/c/curl_multi_timeout.html
|
|
|
|
|
|
| |
Clear the notification before finishing the CANCEL command, so the
notify_wait() after that will always wait for the right notification,
sent by audio_output_all_cancel().
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some users reported that MPD crashes when using a new CURL version
with the threaded DNS resolver enabled. It seems that
curl_multi_fdset() returns no file descriptor when the DNS resolver
runs in another thread, so MPD does not have any event to wait for.
On the CURL mailing list, somebody suggested to sleep for a fixed
amount of time. This is not an elegant solution, because daemons
should never have to sleep without waiting for an event. I hope the
CURL developers will review the API and remove the threaded DNS
resolver.
Meanwhile, I'm removing the assertion in question, to allow those
unfortunate users running the latest CURL version to continue using
MPD.
|
|
|
|
|
| |
Use curl_multi_timeout() to determine the select() timeout, instead of
hard-coding one second.
|
|
|
|
| |
RVA2 tags only store the "gain" value, there is no "peak" attribute.
|
| |
|