aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* playlist: don't preserve "current" song after "random" toggleMax Kellermann2009-02-111-2/+4
| | | | | | | | When MPD is not playing, it may still remember which is the "current" song. When you switch to "random" mode, MPD will always start playing exactly this song. This defies the goal of "random" mode a little. Clear the "current" song when MPD is not playing during the "random" mode switch.
* audio: moved code to output_command.cMax Kellermann2009-02-116-50/+122
| | | | | | The output_command library provides a command interface to the audio outputs. It assumes the input comes from an untrusted source (i.e. the client) and verifies all parameters.
* audio_format: added validation functionsMax Kellermann2009-02-112-6/+37
| | | | | | In addition to audio_format_valid(), provide functions which validate only one attribute of an audio_format. These functions are reused by audio_format_parse().
* audio: replaced parseAudioConfig() with audio_format_parse()Max Kellermann2009-02-116-63/+153
| | | | | | Added audio_format_parse() in a separate library, with a modern interface: return a GError instead of logging errors. This allows the caller to deal with the error.
* Set boundPort to the current portnumber, so zeroconf announces the right ↵Qball Cow2009-02-111-0/+1
| | | | port number
* output_all: immediately reopen output on playMax Kellermann2009-02-101-0/+13
| | | | | | | When MPD explicitly starts playing, ignore the "REOPEN_AFTER" timeout. This timeout was useful when MPD attempted to reopen a failed device over and over, but it confuses users when they explicitly tell MPD to start playing, while MPD insists to wait for the 10 seconds to pass.
* output_thread: moved code to ao_close()Max Kellermann2009-02-101-12/+14
| | | | Merge some duplicate code into one function.
* 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.