aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* output_thread: leave the pause loop on failureMax Kellermann2009-02-101-0/+1
| | | | | When the pause() method fails, leave the pause loop, because calling pause() on a closed device is not allowed.
* output_thread: consistently (de)initialize pcm_convert_stateMax Kellermann2009-02-101-3/+5
| | | | | | Fix a memory leak: it was not guaranteed that pcm_convert_deinit() was called for each pcm_convert_init(). This patch always (de)initializes the pcm_convert library when the audio_output.open flag is flipped.
* output_api: no CamelCase in struct audio_outputMax Kellermann2009-02-104-26/+27
| | | | Renamed audio_output struct members.
* shout_mp3: use audio_format_frame_size()Max Kellermann2009-02-101-2/+1
| | | | | Use audio_format_frame_size() instead of channels*audio_format_sample_size().
* shout: pass void pointer to the encoderMax Kellermann2009-02-103-5/+6
| | | | | | Pass the music chunk as a "const void *" to the encoder, instead of a "const char *". Actually, both encoders currently expect 16 bit samples, passing a 8-bit character is rather pointless.
* shout_ogg: moved PCM conversion to a separate functionMax Kellermann2009-02-101-12/+13
| | | | | | For simplification, moved the PCM conversion code to pcm16_to_ogg_buffer(). Work with a int16_t pointer instead of a char pointer.
* ao: declare AoData.writeSize as size_tMax Kellermann2009-02-101-3/+3
| | | | | writeSize is a memory size and its type should thus be size_t. This allows us to remove two explicit casts.
* osx: removed disabled debug messagesMax Kellermann2009-02-101-43/+2
| | | | Nobody needs these debug messages anymore.
* output_all: added API documentationMax Kellermann2009-02-101-0/+33
|
* output_all: moved code to audio_output_all_finished()Max Kellermann2009-02-101-13/+16
| | | | audio_output_all_finished() returns bool, not int.
* output_all: no CamelCaseMax Kellermann2009-02-104-81/+97
| | | | Renamed functions and variables.
* audio: moved code to output_all.cMax Kellermann2009-02-1010-298/+366
| | | | | Moved code which deals with all audio outputs at once into a separate library.
* audio: moved protocol code to output_print.cMax Kellermann2009-02-106-19/+78
|
* audio: moved state file code to output_state.cMax Kellermann2009-02-106-59/+121
|
* audio: removed isCurrentAudioFormat()Max Kellermann2009-02-102-10/+0
| | | | This function isn't used anymore.
* crossfade: don't use isCurrentAudioFormat()Max Kellermann2009-02-103-2/+5
| | | | | | The crossfading code shouldn't depend on the audio output code. Pass the current audio format to cross_fade_calc() and let it compare directly, instead of using isCurrentAudioFormat().
* playlist: fix assertion when last/current song is deletedMax Kellermann2009-02-101-1/+4
| | | | | | | When MPD is stopped, but the last song is still the "current song", and you delete it, playlist->current is not updated, and becomes an invalid value. Fix this by catching "!playlist->playing && playlist->current == (int)songOrder".
* audio: added output enumeration functionsMax Kellermann2009-02-102-0/+35
| | | | Added audio_output_get(), audio_output_find().
* audio: added audio_output_config_count()Max Kellermann2009-02-102-1/+11
| | | | | | audio_output_config_count() returns the number of audio outputs in the configuration file. It is only used by initAudioDriver(). The public function audio_output_count() now returns audioOutputArraySize.
* output_internal: added missing output_api.h includeMax Kellermann2009-02-101-0/+1
| | | | output_api.h is required for enum audio_output_command.
* player: added some more assertionsMax Kellermann2009-02-102-0/+4
| | | | Assertions on pc.command and pc.next_song.
* player_thread: reset pc.next_song only if queuedMax Kellermann2009-02-101-1/+5
| | | | | | | When we reset pc.next_song if there is no song queued, this might cause a race condition: the next song to be played is cleared, while pc.command was already set. Clear the "next_song" only if there is a song queued for the current do_play() invocation.
* playlist: update queued song after seekingMax Kellermann2009-02-101-3/+7
| | | | | | | | If a new song is queued before calling playerSeek(), then the player and the playlist enter an inconsistent state, because the player discards the playlist's "queued" song in favor of the seeked song. Call playlist_update_queued_song() after playerSeek().
* player_thread: reset pc.next_songMax Kellermann2009-02-101-0/+6
| | | | | | | After a player command (successful or not), reset pc.next_song, because the queue is supposed to be empty then. Otherwise, playlist.queued and pc.next_song may disagree, which triggers an assertion failure.
* queue: update order array after song moveMax Kellermann2009-02-091-0/+14
| | | | | | | Commit f78cddb4 introduced a regression: after a song was moved, the order array was not updated (in random mode). This caused MPD to think the "current" song has changed when you moved something to the position of the current song.
* configure.ac: define HAVE_FFMPEG after all checksMax Kellermann2009-02-092-2/+7
| | | | | | Don't define HAVE_FFMPEG if the ffmpeg libraries were found via pkg-config, but ffmpeg support was disabled because avcodec_decode_audio2() is not available.
* playlist: fixed "next song" checkVladimir S Eremin2009-02-091-3/+2
| | | | Check if the "current+1" position is actually valid.
* output_api: include "config.h" without "../"Max Kellermann2009-02-091-1/+1
| | | | Including "../config.h" breaks on some systems.
* shout: clear buffer before calling the encoderMax Kellermann2009-02-094-18/+15
| | | | | | | Always assume the buffer is empty before calling the encoder. Always flush the buffer immediately after there has been added something. This reduces the risk of buffer overruns, because there will never be a "rest" in the current buffer.
* shout: don't postpone metadataMax Kellermann2009-02-094-34/+15
| | | | | Don't duplicate the tag received by the send_metadata() method - send it to the shout server directly.
* shout: use libshout's synchronizationMax Kellermann2009-02-093-24/+5
| | | | | Removed the manual timer synchronization from the shout plugin. libshout's shout_sync() function does it for us.
* shout: switch to blocking modeMax Kellermann2009-02-093-77/+8
| | | | | | The non-blocking mode of libshout is sparsely documented, and MPD's implementation had several bugs. Also removed connect throttling code, that is done by the MPD core since 0.14.
* shout: removed shout_data.tag_to_sendMax Kellermann2009-02-092-9/+6
| | | | | When shout_data.tag!=NULL, there is a "tag to send". The tag_to_send flag is redundant.
* shout: removed shout_data.shout_errorMax Kellermann2009-02-092-4/+0
| | | | That variable is set in handle_shout_error(), but is never read.
* playlist: provide information about "next song"Vladimir S Eremin2009-02-093-0/+27
| | | | | In random mode, this patch allows clients to see the "next song" in the queue.
* playlist: increase playlist version after loading stateMax Kellermann2009-02-041-0/+2
| | | | | | After the state file has been loaded, the playlist version is still "1", and "plchanges 1" returns the whole playlist. Fix this by increasing the playlist version after the state file has been loaded.
* update: check whether stickers are enabledMax Kellermann2009-02-041-1/+3
| | | | | If stickers are not configured at runtime, don't call sticker_song_delete().
* playlist_save: added spl_save_playlist()Max Kellermann2009-02-043-1/+13
| | | | spl_save_playlist() is a wrapper for spl_save_queue().
* playlist: added G_LOG_DOMAINMax Kellermann2009-02-042-4/+11
| | | | Removed the explicit "playlist:" prefix from all log messages.
* playlist_print: added wrappers for printing the queueMax Kellermann2009-02-043-31/+164
| | | | Hide the details of the playlist behind wrapper functions.
* playlist: pass const playlist pointersMax Kellermann2009-02-042-15/+12
| | | | Pass constant playlist objects to functions which do not modify it.
* playlist_print: added API documentationMax Kellermann2009-02-041-0/+8
|
* playlist_print: use bool instead of intMax Kellermann2009-02-043-16/+22
| | | | | Return true on success, instead of 0. Converted the "detail" parameter to bool.
* playlist_print: no CamelCaseMax Kellermann2009-02-043-5/+6
| | | | Renamed one function.
* playlist_print: include cleanupMax Kellermann2009-02-041-4/+1
|
* playlist: moved code to playlist_edit.cMax Kellermann2009-02-043-369/+394
| | | | | Moved functions for playlist editing (append, delete, shuffle, move) to playlist_edit.c.
* playlist: moved code to playlist_control.cMax Kellermann2009-02-044-241/+306
| | | | | Moved handlers for control commands (play, stop, next, prev) to playlist_control.c.
* playlist: call syncPlaylistWithQueue() only in the event handlerMax Kellermann2009-02-041-4/+0
| | | | | | Don't call syncPlaylistWithQueue() in nextSongInPlaylist() and previousSongInPlaylist(). This is a relic from the time when there was no event, and was a workaround to the timing problem.
* playlist: renamed global "playlist" variable to "g_playlist"Max Kellermann2009-02-0411-391/+494
| | | | | | | Export the "g_playlist" variable, and pass it to all playlist functions. This way, we can split playlist.c easier into separate parts. The code which initializes the singleton variable is moved to playlist_global.c.
* playlist: moved code to playlist_state.cMax Kellermann2009-02-044-134/+217
| | | | | Moved everything related to saving or loading the playlist from/to the state file to playlist_state.c.