aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* song_save: don't call tag_free(NULL)Max Kellermann2009-04-282-1/+4
| | | | | | When a song was in the database twice (which shouldn't happen), and the first song had no tag items, MPD calledd tag_free(NULL). Add a check to that source location, and an assertion to tag_free().
* libsamplerate: use g_ascii_strncasecmp() instead of strncasecmp()Max Kellermann2009-04-281-1/+1
| | | | | strncasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* command: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-1/+1
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* utils: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-1/+1
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* tag: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-282-3/+3
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* conf: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-2/+2
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* locate: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-4/+4
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* input_curl: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-7/+7
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* mad_decoder: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-4/+4
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* mp4ff_decoder: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-8/+9
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* alsa_mixer: use g_ascii_strcasecmp() instead of strcasecmp()Max Kellermann2009-04-281-2/+2
| | | | | strcasecmp() is locale dependent, making it a bad choice for internal string comparisons.
* alsa_mixer: add mixer_index optionDaniel Kahn Gillmor2009-04-281-2/+7
| | | | This allows you to select controls with duplicate names.
* decoder_api: removed dc.pipe assertion from decoder_read()Max Kellermann2009-04-261-1/+0
| | | | | | Database update was broken due to the dc.pipe!=NULL assertion. This assertion is only valid while MPD decodes a song, not during database update.
* vorbis_encoder: reinitialize vorbis_dsp_state after flushMax Kellermann2009-04-251-0/+7
| | | | | | | libvorbis goes into a very long loop if we try to add data after a flush was invoked by vorbis_analysis_wrote(0). This seems to be a problem with the internal end-of-stream marker. Thus, we cannot reuse the vorbis_dsp_state object.
* decoder_control: properly cancel an existing command in dc_stop()Max Kellermann2009-04-251-2/+8
| | | | | | | When the decoder thread has a pending command, send the STOP command to cancel this command. Send STOP again if the decoder thread is still running after that, just in case the decoder thread has executed the previous command (which was overwritten).
* decoder: added assertions on dc.pipeMax Kellermann2009-04-252-0/+13
| | | | | dc.pipe must be non-NULL while the decoder thread is running. Ensure that with a load of assertions.
* client: use GTimer to track timeoutsMax Kellermann2009-04-251-7/+14
|
* playlist_control: use GTimer in previousSongInPlaylist()Max Kellermann2009-04-253-6/+15
| | | | | To determine whether to rewind the current song or to go to the previous song, use a GTimer instead of manually diffing time(NULL).
* pulse_mixer: use PULSE's mainloop lock instead of GMutexMax Kellermann2009-04-251-27/+13
| | | | | | Using two different kinds of locks may result in a race condition with a deadlock. The libpulse callbacks need no locks at all, because the mainloop object can be assumed to be already locked.
* curl: moved proxy settings to "input" blockMax Kellermann2009-04-251-24/+26
| | | | | The old global settings "http_proxy_host", "http_proxy_port", "http_proxy_user" and "http_proxy_password" continue to work.
* fluidsynth: added "experimental" comments in the source fileMax Kellermann2009-04-251-0/+9
|
* player_thread: pause when all audio outputs fail to playMax Kellermann2009-04-251-2/+7
| | | | | | When all audio outputs have been closed due to failures, pause the playback instead of stopping it. This way, the user may resume at the current position after the problem has been dealt with.
* player_thread: pause when output fails while seekingMax Kellermann2009-04-251-7/+11
| | | | | | | | | When no audio outputs could be opened while seeking, leave MPD seeked at that position and pause playback. The user may continue from this point at any time, as soon as the audio outputs are fixed. The old behaviour triggered an assertion failure: the failure wasn't passed properly to the do_play() function, which attempted to play audio chunks.
* stored_playlist: don't close NULL file on errorMax Kellermann2009-04-241-5/+1
| | | | | spl_append_song() can crash when fopen() fails, because it attempts to close the invalid file handle (NULL) in the error handler.
* alsa_output: don't use atexit() to clean up the ALSA libraryMax Kellermann2009-04-211-7/+3
| | | | | Call snd_config_update_free_global() manually in our finish() method, don't use atexit().
* alsa_mixer: call snd_config_update_free_global() in finish()Max Kellermann2009-04-211-1/+3
| | | | | | | | snd_config_update_free_global() frees cached ALSA configuration. This keeps valgrind a little bit more quiet. This patch moves the call from the open() method into the finish() method, which seems more natural: it allows the use of the config cache, and improves the cleanup phase.
* output_control: close mixer on pauseMax Kellermann2009-04-211-0/+6
| | | | | For non-global mixers (only "pulse" currently), close the mixer when MPD playback is paused.
* client: free the fifo bufferMax Kellermann2009-04-171-0/+2
| | | | | This patch fixes a memory leak: the fifo_buffer object was not freed when the client connection was closed.
* icy_server: don't export icy_server_metadata_string()Max Kellermann2009-04-132-4/+1
| | | | | icy_server_metadata_string() is only called by icy_server_metadata_page().
* httpd: use C99 "bool" instead of GLib's gbooleanMax Kellermann2009-04-131-7/+7
| | | | Plain "bool" consumes only one byte instead of four.
* httpd: use g_ascii_strncasecmp() to compare headersMax Kellermann2009-04-131-1/+1
| | | | In HTTP, header names are case insensitive.
* Implemented basic icy support for the httpd outputHagen Schink2009-04-136-12/+409
| | | | | | [mk: folded with patch "Put icy related functions in extra source files"; moved icy_server.c from HAVE_CURL to ENABLE_HTTPD_OUTPUT; removed an unused variable]
* decoder_api: submit the song tag to the music pipeMax Kellermann2009-04-133-6/+24
| | | | | | When a new song starts playing, send its tag (song->tag) to the music pipe. This allows output plugins to render tags for all songs, not only those with embedded tags understood by the decoder plugin.
* log: removed "unused" attribute from log_level parameterMax Kellermann2009-04-101-1/+1
|
* decoder: prefer wildmidi over fluidsynthAvuton Olrich2009-04-061-4/+4
|
* vorbis: move #define out of function to top of sourcesAvuton Olrich2009-04-021-1/+1
|
* decoder: Rename all main decoder plugins functions to *decoder_plugin.Avuton Olrich2009-04-025-12/+12
|
* configure.ac: renamed --enable-mod to --enable-mikmodMax Kellermann2009-04-011-1/+1
|
* httpd: don't pass uninitialized page to httpd_client_check_queue()Max Kellermann2009-04-011-1/+1
| | | | | The httpd_client_check_queue() callback function does not use its "user_data" argument. Don't pass any, and fix the gcc warning.
* configure.ac: renamed --enable-oggvorbis to --enable-vorbisMax Kellermann2009-04-011-1/+1
|
* command: added command "sticker find"Max Kellermann2009-04-011-0/+41
| | | | | This command allows clients to search for stickers with a specified name.
* Move db_get_song into sub-handlers. (FReq 2112)Eric Wollesen2009-04-011-9/+27
| | | | | | | | db_get_song was being called once for all sub-handlers, but with the addition of the find command, we don't have a URI coming in, so doing db_get_song once won't work anymore. [mk: fixed initialization order]
* song_sticker: added song_sticker_find()Max Kellermann2009-04-012-0/+76
| | | | This is a wrapper for sticker_find(), which looks up the song object.
* sticker: added sticker_find()Max Kellermann2009-04-012-0/+85
| | | | sticker_find() finds stickers with the specified name.
* directory: added directory_lookup_song()Max Kellermann2009-04-013-25/+41
| | | | Moved code from db_get_song().
* directory: renamed directory_get_directory()Max Kellermann2009-04-013-6/+13
| | | | | Renamed directory_get_directory() to directory_lookup_directory(). Added API documentation.
* sticker_print: new library for sending stickers to a clientMax Kellermann2009-04-013-10/+85
|
* sticker: pass const sticker to sticker_foreach()Max Kellermann2009-04-012-4/+4
|
* client, event_pipe: explicitly ignore the write() resultMax Kellermann2009-04-012-2/+2
| | | | | | | | | | On both locations, the result of write() can be ignored safely. In event_pipe_emit_fast(), that can only be "EAGAIN", which means that the pipe buffer is full - no further notification required. In client_init(), that would be a fatal connection error, which would be caught by the next event. This patch fixes gcc warnings.
* socket_util: fixed format warningMax Kellermann2009-04-011-1/+1
| | | | | g_set_error() is a printf-like function, and expects a format string. Using the return value of gai_strerror() is unsafe.