Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | directory: don't require a "directory" visitor | Max Kellermann | 2011-09-13 | 1 | -6/+6 | |
| | | | | | | | | | | .. to recurse. A visitor may not have one, but still be interested in a recursive search. | |||||
* | | database: add struct db_visitor | Max Kellermann | 2011-09-13 | 12 | -196/+361 | |
| | | | | | | | | Use this struct for db_walk(). | |||||
* | | stored_playlist: return GError, code is playlist_result | Max Kellermann | 2011-09-11 | 10 | -197/+320 | |
| | | | | | | | | | | Improve error reporting and handling. command.c gets the new function print_error(), which sends a GError to the client. | |||||
* | | playlist: move enum playlist_result to playlist_error.h | Max Kellermann | 2011-09-11 | 8 | -17/+49 | |
| | | | | | | | | Reduce header dependencies. | |||||
* | | playlist: move PLAYLIST_COMMENT to stored_playlist.c | Max Kellermann | 2011-09-11 | 2 | -2/+2 | |
| | | | | | | | | Only used there. | |||||
* | | db_plugin: introducing a plugin API for the song database | Max Kellermann | 2011-09-10 | 7 | -189/+527 | |
| | | | | | | | | | | | | | | | | | | | | | | First draft, not really pluggable currently - hard-coded to use the "simple" plugin, and calls several of its internal functions. The API is very simple currently, all searches are still performed over the root "directory" object. Future changes to the API will move those search implementations into the plugin, to allow more efficient implementations, or implementations that don't have the whole tree in memory all the time. | |||||
* | | database: move code to db_save.c | Max Kellermann | 2011-09-10 | 3 | -128/+214 | |
| | | ||||||
* | | directory_save: pass const pointer to _save() | Max Kellermann | 2011-09-10 | 2 | -4/+4 | |
| | | ||||||
* | | tag_internal: add missing tag.h include | Max Kellermann | 2011-09-10 | 1 | -0/+2 | |
| | | | | | | | | For TAG_NUM_OF_ITEM_TYPES. | |||||
* | | conf: export config_param_free() | Max Kellermann | 2011-09-09 | 2 | -1/+4 | |
| | | ||||||
* | | conf: _get_next_param() returns a const pointer | Max Kellermann | 2011-09-09 | 2 | -3/+3 | |
| | | | | | | | | No writers. | |||||
* | | database: return GError on failure | Max Kellermann | 2011-09-09 | 4 | -23/+41 | |
| | | ||||||
* | | conf: add config_dup_block_path() | Max Kellermann | 2011-09-09 | 3 | -14/+38 | |
| | | ||||||
* | | utils: parsePath() returns GError on failure | Max Kellermann | 2011-09-09 | 4 | -14/+30 | |
| | | | | | | | | Better error messages. | |||||
* | | utils: pass a const string to parsePath() | Max Kellermann | 2011-09-09 | 2 | -15/+14 | |
| | | | | | | | | Remove the slash hack, allocate memory for the user name. | |||||
* | | utils: eliminate local variable "pos" | Max Kellermann | 2011-09-09 | 1 | -3/+4 | |
| | | | | | | | | Just advance the "path" pointer. | |||||
* | | conf: turn config_get_path() into config_dup_path() | Max Kellermann | 2011-09-09 | 4 | -47/+109 | |
| | | | | | | | | | | | | | | | | | | config_get_path() was somewhat flawed, because it pretended to be a function, when it really had a side effect. The second flaw was that it did not return the parser error, instead it aborted the whole process, which is bad style. The new function returns a duplicated (modified) string that must be freed by the caller, and returns a GError on failure. | |||||
* | | glib_compat.h: emulate g_prefix_error() | Max Kellermann | 2011-09-09 | 1 | -0/+6 | |
| | | ||||||
* | | log: print detailed errno message on open() failure | Max Kellermann | 2011-09-09 | 1 | -2/+2 | |
| | | ||||||
* | | log: return GError on initialization failure | Max Kellermann | 2011-09-09 | 3 | -11/+34 | |
| | | ||||||
* | | conf: get_block_param() returns a const pointer | Max Kellermann | 2011-09-09 | 3 | -6/+6 | |
| | | | | | | | | No caller needs to write. | |||||
* | | conf: move duplicate check to _read_name_value() | Max Kellermann | 2011-09-09 | 2 | -16/+15 | |
| | | | | | | | | config_add_block_param() cannot fail, which makes it easier to use. | |||||
* | | conf: move code to config_read_name_value() | Max Kellermann | 2011-09-09 | 1 | -32/+34 | |
| | | | | | | | | Reduce indent. | |||||
* | | conf: add gcc optimization attributes | Max Kellermann | 2011-09-09 | 1 | -0/+3 | |
| | | ||||||
* | | dbUtils: move code to db_print.c | Max Kellermann | 2011-09-06 | 5 | -269/+317 | |
| | | ||||||
* | | dbUtils: pass player_control to findAddIn() | Max Kellermann | 2011-09-05 | 3 | -9/+14 | |
| | | | | | | | | Allow calling findAddIn() without a client object. | |||||
* | | decoder/oggflac: delete this obsolete plugin | Max Kellermann | 2011-09-01 | 2 | -355/+1 | |
| | | | | | | | | | | libOggFLAC has been deprecated for 5 years now, and we havn't been testing it for a long time. Let's delete it. | |||||
* | | Merge branch 'v0.16.x' | Max Kellermann | 2011-09-01 | 9 | -56/+95 | |
|\| | | | | | | | | | | | Conflicts: configure.ac src/output_control.c | |||||
| * | output/osx: don't drain the buffer when closing | Max Kellermann | 2011-09-01 | 1 | -6/+0 | |
| | | | | | | | | Eliminate an unnecessary source of deadlocks. | |||||
| * | output/osx: signal the GCond while mutex is locked | Max Kellermann | 2011-09-01 | 1 | -1/+1 | |
| | | ||||||
| * | output_all: move _lock_signal() to output_control.c | Max Kellermann | 2011-09-01 | 3 | -21/+23 | |
| | | | | | | | | Better name, better documentation. | |||||
| * | output_thread: reimplement CANCEL synchronization | Max Kellermann | 2011-09-01 | 6 | -14/+34 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | |||||
| * | output/pulse: reset callbacks before closing stream/context | Max Kellermann | 2011-08-31 | 1 | -0/+10 | |
| | | | | | | | | | | Fixes assertion failure when a stream callback is invoked too late after a format change. | |||||
| * | output/pulse: add function _delete_stream() | Max Kellermann | 2011-08-31 | 1 | -5/+16 | |
| | | | | | | | | Merge common code. | |||||
| * | output/pulse: use _delete_context() | Max Kellermann | 2011-08-31 | 1 | -16/+15 | |
| | | | | | | | | Eliminate duplicate code. | |||||
| * | input/curl: implement a hard-coded timeout of 10 seconds | Max Kellermann | 2011-08-26 | 1 | -0/+3 | |
| | | | | | | | | | | Be sure to stop the operation at some point when the server isn't responding. | |||||
* | | output/raop: remove Audio-Jack-Status check | Max Kellermann | 2011-08-31 | 2 | -27/+7 | |
| | | | | | | | | | | The value of this is not used, the code is commented out - let's get rid of it. | |||||
* | | output/raop: remove empty function raopcl_stream_connect() | Max Kellermann | 2011-08-31 | 1 | -8/+0 | |
| | | ||||||
* | | rtsp_client: use the I/O thread | Max Kellermann | 2011-08-31 | 4 | -56/+615 | |
| | | | | | | | | Make the code portable. | |||||
* | | udp_server: don't use MSG_DONTWAIT on WIN32 | Max Kellermann | 2011-08-31 | 1 | -0/+4 | |
| | | | | | | | | Doesn't exist on mingw32. | |||||
* | | mixer/raop: include cleanup | Max Kellermann | 2011-08-31 | 1 | -12/+2 | |
| | | ||||||
* | | output/raop: cast sendto() parameter to void pointer | Max Kellermann | 2011-08-31 | 1 | -2/+4 | |
| | | | | | | | | Fix compilation on mingw32. | |||||
* | | rtsp_client: use g_usleep() | Max Kellermann | 2011-08-31 | 1 | -3/+1 | |
| | | | | | | | | usleep() is not portable. | |||||
* | | rtsp_client: allow parameter "kd" to be NULL | Max Kellermann | 2011-08-31 | 3 | -8/+17 | |
| | | | | | | | | When the caller isn't interested in the values. | |||||
* | | rtsp_client: don't wait if packet is over due | Max Kellermann | 2011-08-31 | 1 | -2/+3 | |
| | | | | | | | | | | Check if the time difference is negative. That would have caused sleeping forever. | |||||
* | | io_thread: allow _call() from inside the thread | Max Kellermann | 2011-08-31 | 2 | -0/+20 | |
| | | ||||||
* | | rtsp_client: free attribute "kd" in _close() | Max Kellermann | 2011-08-31 | 1 | -0/+1 | |
| | | | | | | | | Fix yet another memory leak. | |||||
* | | output/raop: move code to raop_session_new() | Max Kellermann | 2011-08-30 | 1 | -40/+74 | |
| | | | | | | | | .. and fix a few memory leaks. | |||||
* | | output_control: move code to audio_output_destruct() | Max Kellermann | 2011-08-30 | 3 | -17/+57 | |
| | | | | | | | | .. and destruct the output object properly in test/run_output. | |||||
* | | output/raop: merge raopcl_close() into _finish() | Max Kellermann | 2011-08-30 | 1 | -10/+4 | |
| | | | | | | | | .. and fix a double free bug. |