| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Since we are using curl_multi_info_read() / CURLMSG_DONE for detecting
end-of-response, we can remove all running_handles==0 checks. For
some reason, that has never worked correctly.
|
|
|
|
|
|
|
| |
curl_multi_info_read() is the authoritative source of the
"end-of-response" information. Always set c->eof when a CURLMSG_DONE
message is received, and check the result (success/failure) after
that.
|
|
|
|
|
|
| |
Define the special value "-1" as "unknown size". Previously, there
was no indicator for streams with unknown size, which might confuse
some decoders.
|
|
|
|
|
| |
libcurl provides better error messages than curl_multi_strerror() when
you set the ERRORBUFFER option.
|
|
|
|
|
|
| |
The read() method must not return a negative value. Make it return 0
on error, just like the file input stream. Also set the "error"
attribute to -1.
|
|
|
|
|
|
|
| |
I had this option enabled during development, but at some point, it
must have gotten lost. FAILONERROR makes the curl stream fail when
the server returns a status code 400 or higher. We are not interested
in the server's error document.
|
|
|
|
|
| |
The function curl_multi_info_read() provides access to errors from the
curl easy interface.
|
|
|
|
|
|
|
|
| |
During codec detection, the beginning of the stream is consumed. This
is a common operation, which takes a lot of time when handling remote
resources. To optimize this, remember the first 64 kB of a stream.
This way, we can rewind the stream without actually fetching the start
of the stream again.
|
|
|
|
|
|
| |
size_t and long aren't 64 bit safe (i.e. files larger than 2 GB on a
32 bit OS). Use off_t instead, which is a 64 bit integer if compiled
with large file support.
|
|
|
|
|
| |
Don't attempt to open a HTTP URL as a local file, and don't send a
local path to libcurl.
|
|
|
|
|
| |
If the server sends the headers "icy-name", "ice-name",
"x-audiocast-name", set the stream title.
|
| |
|
|
|
|
|
|
| |
Replaced the local variable "colon" (which had only temporary meaning)
with the variable "value". It is a pointer to the first byte of the
header value.
|
|
|
|
|
| |
For boolean values and success flags, use bool instead of integer (1/0
for true/false, 0/-1 for success/failure).
|
|
|
|
| |
close() shouldn't fail with read-only streams.
|
|
|
|
|
|
|
| |
Instead of managing a set of method pointers in each input_stream
struct, move these into the new input_plugin struct. Each
input_stream has only a pointer to the plugin struct. Pointers to all
implementations are kept in the array "input_plugins".
|
|
|
|
| |
Renamed inputStream.c and inputStream_file.c.
|
|
MPD's HTTP client code has always been broken, no matter how effort
was put into fixing it. Replace it with libcurl, which is known to be
quite stable. This adds a fat library dependency, but only for people
who need streaming.
|