Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | decoder_thread: fix assertion failure at song end | Max Kellermann | 2011-01-16 | 1 | -2/+1 |
| | | | | | | Don't finish the current command twice. This bug was never noticed, but was revealed by a new assertion check. | ||||
* | output_control: document internal functions | Max Kellermann | 2011-01-10 | 1 | -0/+17 |
| | |||||
* | output_control: move code to ao_lock_command() | Max Kellermann | 2011-01-10 | 1 | -9/+15 |
| | |||||
* | output_control: ao_command() calls ao_command_async() | Max Kellermann | 2011-01-10 | 1 | -7/+6 |
| | | | | Merge some code. | ||||
* | update_remove: use plain GCond instead of notify | Max Kellermann | 2011-01-10 | 1 | -9/+16 |
| | | | | GMutex/GCond guarantee that the access to removed_song is protected. | ||||
* | include cleanup | Max Kellermann | 2011-01-10 | 3 | -1/+4 |
| | |||||
* | decoder_control: store GCond object, not a player_control | Max Kellermann | 2011-01-10 | 7 | -28/+20 |
| | | | | | | Remove the decoder dependency on player_control. All player_control was needed for is to signal the player thread, and we can do that with a simple GCond as well. | ||||
* | decoder_thread: move code to decoder_command_finished_locked() | Max Kellermann | 2011-01-10 | 1 | -8/+19 |
| | |||||
* | decoder_control: remove unused function dc_command_wait() | Max Kellermann | 2011-01-10 | 2 | -11/+0 |
| | | | | Only dc_command_wait_locked() is really being used. | ||||
* | decoder_control: replace dc_init() with dc_new() | Max Kellermann | 2011-01-10 | 3 | -15/+17 |
| | | | | | dc_new() allocates the object and returns it. dc_free() frees it (replaces dc_deinit()). | ||||
* | player_control: removed the global variable "pc" | Max Kellermann | 2011-01-10 | 41 | -571/+738 |
| | | | | | | | Allocate a player_control object where needed, and pass it around. Each "client" object is associated with a "player_control" instance. This prepares multi-player support. | ||||
* | dbUtils: don't use directoryAddSongToPlaylist() twice | Max Kellermann | 2011-01-10 | 1 | -1/+1 |
| | | | | | In findAddInDirectory(), call playlist_append_song() directly, to have some more type checking. | ||||
* | input_init: add assertions on completeness of plugins | Max Kellermann | 2011-01-08 | 1 | -0/+6 |
| | |||||
* | input/cdio_paranoia: add module name | Max Kellermann | 2011-01-08 | 1 | -0/+1 |
| | | | | | Seems like we forgot to give this module a name... this can crash MPD on startup. | ||||
* | Merge branch 'v0.16.x' | Max Kellermann | 2011-01-07 | 10 | -57/+65 |
|\ | | | | | | | | | | | Conflicts: NEWS configure.ac | ||||
| * | Merge branch 'v0.15.x' into v0.16.x | Max Kellermann | 2011-01-07 | 3 | -2/+12 |
| |\ | | | | | | | | | | | | | | | | | | | Conflicts: NEWS configure.ac src/directory.h | ||||
| | * | encoder/lame: explicitly configure the output sample rate | Max Kellermann | 2011-01-07 | 1 | -0/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
| | * | output/httpd: define G_LOG_DOMAIN in httpd_client.c | Max Kellermann | 2011-01-07 | 1 | -0/+3 |
| | | | |||||
| | * | directory: fix warning "comparison between signed and unsigned" | Max Kellermann | 2010-12-21 | 1 | -2/+2 |
| | | | | | | | | | | | | Cast the constant to dev_t, not to unsigned. | ||||
| | * | zeroconf-bonjour: use g_htons() instead of htons() | Max Kellermann | 2010-12-21 | 1 | -1/+1 |
| | | | | | | | | | | | | Fixes the gcc warning "implicit declaration of function 'htons'". | ||||
| * | | player_thread: discard empty chunks while cross-fading | Max Kellermann | 2011-01-07 | 1 | -0/+13 |
| | | | | | | | | | | | | | | | | | | When a music_chunk to be crossfaded consists only of a tag, cross-fading is not possible, and led to an assertion failure. This patch just discards those, as if cross-fading was not enabled. | ||||
| * | | output_thread: fix double lock | Max Kellermann | 2011-01-07 | 1 | -3/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | During the whole output thread, the audio_output object is locked, and it is only unlocked while waiting for the GCond and while running a plugin method. The error handler in ao_play_chunk() attempted to lock the object again, which was code from MPD 0.15.x which should have been removed a long time ago. | ||||
| * | | player_thread: fix assertion failure due to early seek | Max Kellermann | 2011-01-07 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | | | | | | Until the decoder plugin has called decoder_initialized(), the player may not submit seek commands. This however could occur with a slow decoder and a CUE file with a virtual song offset. This patch adds another check. | ||||
| * | | player_thread: make variables more local | Max Kellermann | 2011-01-07 | 1 | -36/+19 |
| | | | |||||
| * | | output/osx: fix up audio format first, then apply it to device | Max Kellermann | 2011-01-07 | 1 | -9/+10 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a MPD 0.16 regression: when playing a 24 bit file, the switch to 16 bit was made only partially, after mBytesPerPacket and mBytesPerFrame had already been applied. That means mBytesPerFrame referred to 24 bit, and mBitsPerChannel referred to 16 bits. Of course, that cannot work. | ||||
| * | | add void casts to suppress "result unused" warnings (clang) | Max Kellermann | 2010-12-21 | 2 | -3/+3 |
| | | | |||||
| * | | decoder/mad: work around build failure on Solaris | Alex Viskovatoff | 2010-12-21 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | Rename the "version" struct, because it seems to be a reserved name on Solaris: "src/decoder/mad_decoder_plugin.c", line 550: (enum) tag redeclared: version cc: acomp failed for src/decoder/mad_decoder_plugin.c | ||||
| * | | output/solaris: add missing parameter to open_cloexec() call | Alex Viskovatoff | 2010-12-21 | 1 | -1/+1 |
| | | | |||||
| * | | audio_check: fix parameter in prototype | Alex Viskovatoff | 2010-12-21 | 1 | -1/+1 |
| | | | |||||
* | | | Build fix for Linux. Define _GNU_SOURCE in src/server_socket.c. | Ben Kibbey | 2010-12-27 | 1 | -0/+5 |
| | | | |||||
* | | | stored_playlist: remove database lookups from spl_load() | Max Kellermann | 2010-12-23 | 1 | -7/+1 |
| | | | | | | | | | | | | Don't look up songs in the database, no caller needs this. | ||||
* | | | stored_playlist: ignore empty lines | Max Kellermann | 2010-12-23 | 1 | -1/+1 |
| | | | |||||
* | | | stored_playlist: use the text_file library to read lines | Max Kellermann | 2010-12-23 | 1 | -6/+4 |
| | | | |||||
* | | | playlist_song: add flag "secure" | Max Kellermann | 2010-12-23 | 6 | -14/+24 |
| | | | | | | | | | | | | | | | | | | Optionally allow all local files. "Insecure" mode is used for printing playlists. | ||||
* | | | string_util: add function strchug_fast() | Max Kellermann | 2010-12-23 | 7 | -10/+64 |
| | | | | | | | | | | | | Replace g_strchug() calls with a cheaper implementation. | ||||
* | | | utils: move string_array_contains() to string_util.c | Max Kellermann | 2010-12-23 | 8 | -27/+78 |
| | | | |||||
* | | | decoder: new "pcm" decoder plugin | Max Kellermann | 2010-12-22 | 3 | -0/+119 |
| | | | | | | | | | | | | Just there to support the "cdio_paranoia" input plugin. | ||||
* | | | configure.ac: hook the cdio_paranoia input plugin | Max Kellermann | 2010-12-22 | 2 | -0/+10 |
| | | | |||||
* | | | input/cdda: rename plugin to "cdio_paranoia" | Max Kellermann | 2010-12-22 | 2 | -42/+42 |
| | | | |||||
* | | | input/cdda: support reading the whole disc | Max Kellermann | 2010-12-22 | 1 | -10/+8 |
| | | | |||||
* | | | input/cdda: remove unused function input_cdda_archive_extract_trackno() | Max Kellermann | 2010-12-22 | 1 | -24/+0 |
| | | | |||||
* | | | input/cdda: import missing pcm16_to_wave() from the wave encoder | Max Kellermann | 2010-12-22 | 1 | -0/+11 |
| | | | |||||
* | | | input/cdda: implement parse_cdda_uri() | Max Kellermann | 2010-12-22 | 1 | -10/+59 |
| | | | | | | | | | | | | Was missing. | ||||
* | | | input/cdda: call input_cdda_close() with an input_stream pointer | Max Kellermann | 2010-12-22 | 1 | -5/+5 |
| | | | | | | | | | | | | Another build fix. | ||||
* | | | input/cdda: move code to cdda_detect_drive() | Max Kellermann | 2010-12-22 | 1 | -7/+18 |
| | | | |||||
* | | | input/cdda: remove newlines from error messages | Max Kellermann | 2010-12-22 | 1 | -8/+10 |
| | | | |||||
* | | | input/cdda: remove debug code | Max Kellermann | 2010-12-22 | 1 | -31/+1 |
| | | | |||||
* | | | input/cdda: some general build failure fixes | Max Kellermann | 2010-12-22 | 1 | -13/+5 |
| | | | |||||
* | | | input_cdda_plugin: new plugin to handle cdda:// protocol to access CD-DA ↵ | Viliam Mateicka | 2010-12-22 | 2 | -0/+406 |
| | | | | | | | | | | | | audio CD's | ||||
* | | | osx_output: always pass OS X result code to g_set_error(). | Greg Ward | 2010-12-21 | 1 | -9/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Should be safe on OS X 10.4 (32-bit), since Apple's OSStatus boils down to "signed long", and g_set_error() takes gint, which is really just "int". Assigning "signed long" to "int" on 32-bit Unix should be just fine, since both are signed 32-bit ints. No idea if this is safe on 64-bit OS X. |