aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* input_stream: non-blocking I/OMax Kellermann2011-09-1657-377/+653
| | | | | | | | | Add GMutex, GCond attributes which will be used by callers to conditionally wait on the stream. Remove the (now-useless) plugin method buffer(), wait on GCond instead. Lock the input_stream before each method call. Do the same with the playlist plugins.
* input_plugin: add method check()Max Kellermann2011-09-167-0/+78
| | | | | To check for errors without reading. The decoder thread wants to do that, before it passes the input stream to the plugin.
* input/{soup,curl}: free unused postponed_errorMax Kellermann2011-09-162-0/+8
| | | | Fix memory leak.
* input/soup: move libsoup calls to the I/O threadMax Kellermann2011-09-161-4/+25
| | | | | libsoup's asynchronous API is not thread safe. By moving the calls into the I/O thread, several crash bugs will be fixed.
* input/soup: wait for _cancel_message() to completeMax Kellermann2011-09-161-5/+21
| | | | | Add the "complete" attribute, and set it in _session_callback(). _close() waits for it to become true.
* input/soup: return real GErrors to the callerMax Kellermann2011-09-161-3/+31
| | | | | Add attribute postponed_error, pass this GError to the caller.
* input/soup: set the "ready" flag on failureMax Kellermann2011-09-161-0/+2
| | | | Fix deadlock.
* input/soup: remove redundant flag "ready"Max Kellermann2011-09-161-6/+2
| | | | | We'll add proper locking now, and there's no need to carry this duplicate flag.
* input/soup: report failure in _read()Max Kellermann2011-09-161-0/+1
| | | | To report failures to callers who don't use _buffer().
* input/soup: unlock before cancelling the soup messageMax Kellermann2011-09-161-3/+5
| | | | Avoid recursive deadlock in _session_callback().
* decoder_api: remove the loop from decoder_read()Max Kellermann2011-09-161-16/+10
| | | | | This was useless, because input_stream_read() blocks until data is available. Calling it in a loop doesn't make sense.
* decoder_api: move code to decoder_check_cancel_read()Max Kellermann2011-09-161-12/+28
|
* input/curl: wait some more before resuming the streamMax Kellermann2011-09-161-1/+6
| | | | | Pausing and resuming after every little chunk adds lots of overhead. Add a lower level for resuming the stream.
* input/curl: clear the "paused" flag before resumingMax Kellermann2011-09-161-1/+1
| | | | | curl_easy_pause(CURLPAUSE_CONT) may have added enough data to go into pause again.
* input/curl: remove _schedule_update()Max Kellermann2011-09-161-48/+1
| | | | | Call _update_fds() directly. This is possible because it's only called from within the I/O thread.
* input/curl: unlock the mutex for io_thread_call()Max Kellermann2011-09-161-1/+4
| | | | Fix deadlock.
* test/test_queue_priority: work around "set but not used" warningMax Kellermann2011-09-161-0/+1
|
* input/soup: fix "unused local variable" warningsMax Kellermann2011-09-161-1/+1
|
* Merge branch 'v0.16.x'Max Kellermann2011-09-165-7/+11
|\
| * configure.ac: disable assertions in the non-debugging buildMax Kellermann2011-09-162-4/+4
| | | | | | | | Add -DNDEBUG to AM_CPPFLAGS.
| * input/curl, output/pulse: fix "unused local variable" warningsMax Kellermann2011-09-162-2/+3
| |
| * configure.ac: enable -Werror for C++Max Kellermann2011-09-161-0/+1
| |
| * configure.ac: pass AM_CXXFLAGS, AM_CPPFLAGS to Makefile.amMax Kellermann2011-09-162-1/+3
| |
* | input_stream: add "nonnull" attributesMax Kellermann2011-09-151-0/+12
| |
* | Merge branch 'v0.16.x'Max Kellermann2011-09-154-11/+44
|\| | | | | | | | | | | Conflicts: NEWS configure.ac
| * decoder/ffmpeg: flush the codec after seekingMax Kellermann2011-09-151-1/+3
| | | | | | | | | | Let the codec start with fresh buffers. This should fix the remaining seeking issues.
| * decoder/ffmpeg: explicitly specify the current stream for seekingMax Kellermann2011-09-151-2/+11
| | | | | | | | | | Use AVStream.time_base to convert the decoder_seek_where() value, and pass the current stream number to av_seek_frame().
| * decoder/ffmpeg: don't require key frame for seekingMax Kellermann2011-09-152-1/+3
| | | | | | | | Use flag AV_TIME_BASE.
| * decoder/ffmpeg: higher precision timestampsMax Kellermann2011-09-152-1/+4
| |
| * decoder/ffmpeg: move formula to time_from_ffmpeg()Max Kellermann2011-09-151-2/+10
| |
| * decoder/ffmpeg: add local variable "av_stream"Max Kellermann2011-09-151-3/+4
| | | | | | | | Code simplification.
| * input/rewind: copy the MIME type only onceMax Kellermann2011-09-152-1/+5
| | | | | | | | Reduce heap usage by reducing the number of malloc() / free() calls.
| * .gitignore: add doxygen.confMax Kellermann2011-09-091-0/+1
| |
| * pcm_format: fix 32-to-24 bit conversion (the "silence" bug)Max Kellermann2011-09-082-2/+3
| | | | | | | | | | | | 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).
| * Modify version string to post-release version 0.16.5~gitAvuton Olrich2011-09-012-1/+4
| |
* | input/curl: per-request mutex/condMax Kellermann2011-09-151-33/+35
| | | | | | | | | | | | | | The global data structures are now lock-free, because they are accessed only from the I/O thread. By using per-request locks, we have finer grained locking, preparing for locks shared with the client.
* | input/curl: move all libCURL calls to the I/O threadMax Kellermann2011-09-151-29/+69
| | | | | | | | | | | | This adds some overheads for indirect calls to the I/O thread, but reduces the amount of global locks. Next step will be switching to per-request locks.
* | input/file: return NULL instead of "false"Max Kellermann2011-09-152-5/+5
| |
* | Makefile.am: allow out-of-tree sparse callMax Kellermann2011-09-151-1/+2
| |
* | input/curl: don't set "ready" on _easy_free()Max Kellermann2011-09-151-3/+2
| | | | | | | | | | Move the assignment to _request_done(), right before the GCond is signalled. This might fix spurious wakeups.
* | input/curl: add assertions on io_thread_inside()Max Kellermann2011-09-151-0/+9
| |
* | input/curl: merge _request_abort() into _request_done()Max Kellermann2011-09-151-31/+19
| | | | | | | | | | This is a trivial function. Merge some duplicate code, e.g. the g_cond_broadcast() call.
* | udp_server: disable -Wmissing-field-initializersMax Kellermann2011-09-151-0/+7
| |
* | gcc.h: add macro GCC_CHECK_VERSIONMax Kellermann2011-09-151-1/+6
| |
* | input/curl: API documentation fixMax Kellermann2011-09-151-1/+1
| |
* | input_plugin: add method update()Max Kellermann2011-09-144-0/+33
| | | | | | | | | | Update the struct attributes, important for facades like the "rewind" plugin. To replace buffer().
* | input_internal: add assertionsMax Kellermann2011-09-141-0/+9
| |
* | input_stream: move input_stream_init(), _deinit() to _internal.cMax Kellermann2011-09-1415-20/+89
| |
* | decoder_internal: don't call input_stream_buffer()Max Kellermann2011-09-143-37/+12
| | | | | | | | | | | | This is not necessary since all relevant input plugins have been moved to the I/O thread, and there is no remaining useful buffer() implementation. This also fixes a busy loop when playing radio.
* | test/read_tags: initialize the tag_pool libraryMax Kellermann2011-09-131-0/+5
| |