aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* eliminate g_error() usageThomas Jansen2010-09-2527-103/+166
| | | | | | | | | | | | | | Replaced all occurrences of g_error() with MPD_ERROR() located in a new header file 'mpd_error.h'. This macro uses g_critical() to print the error message and then exits gracefully in contrast to g_error() which would internally call abort() to produce a core dump. The macro name is distinctive and allows to find all places with dubious error handling. The long-term goal is to get rid of MPD_ERROR() altogether. To facilitate the eventual removal of this macro it was added in a new header file rather than to an existing header file. This fixes #2995 and #3007.
* output/httpd: bind_to_address support (including IPv6)Thomas Jansen2010-09-251-12/+52
| | | | | | | | | Added support for a new optional configuration setting for the httpd output named "bind_to_address". Setting it to a specific IP address (v4 or v6) will cause the httpd output to bind to that address exclusively. Supporting multiple addresses in parallel is future work. This implements the feature requests #2998 and #2646.
* conf: Whitespace cosmeticsThomas Jansen2010-09-251-9/+9
|
* playlist: fix "queued" check in playlist_sync()Max Kellermann2010-09-231-1/+1
| | | | | The check was meant to fix an assertion failure, but it was the wrong way around. This broke cross-fading most of the time.
* decoder/mp4ff: support more variations of "album artist"Max Kellermann2010-09-231-0/+4
| | | | | | | | | | | According to the mantis bug report 2847, there are several possible variations of the "album artist" tag: - "album artist" - "album_artist" - "albumartist" This patch adds support for the latter two.
* listen: fix "unused parameter" warning on WIN32Max Kellermann2010-09-231-0/+2
|
* main: Add Windows Service supportDenis Krjuchkov2010-09-235-1/+228
| | | | | | | | | I've added PIPE_EVENT_SHUTDOWN because calling g_main_loop_quit() do not work when called from another thread. Main thread was sleeping in g_poll() so I needed some way to wake it up. By some strange reason call close(event_pipe[0]) in event_pipe_deinit() hangs. In current implementation that code never reached so that was not a problem :-) I've added a conditional to leave event_pipe[0] open on Win32.
* playlist_state: Fix the "state" line in the outputThomas Jansen2010-09-231-1/+1
| | | | | | | An '\n' was erroneously inserted in the line containing the state, e.g. "state: \nplay" instead of "state: play". Fix for bug #2992.
* zeroconf-bonjour: use g_htons() instead of htons()Max Kellermann2010-09-231-1/+1
| | | | htons() is not available if netinet/in.h is not included.
* directory: cast DEVICE_INARCHIVE, DEVICE_CONTAINER to dev_tMax Kellermann2010-09-231-2/+2
| | | | Fix gcc warning.
* playlist_database: initialize pm.mtimeMax Kellermann2010-09-231-1/+3
|
* fd_util: work around aliasing warning in recvmsg_cloexec()Max Kellermann2010-09-231-2/+2
|
* output/httpd: access sockaddr_storage object directlyMax Kellermann2010-09-231-3/+2
| | | | Work around aliasing warning.
* decoder_control: use g_free() to manage mixramp allocationsMax Kellermann2010-09-233-20/+10
| | | | | Be consistent with the rest of MPD, and don't use the non-portable header "malloc.h".
* update_walk: update existing playlist entryMax Kellermann2010-09-071-1/+3
| | | | Fixes duplicate playlist entries.
* playlist_vector: update_or_add() returns boolMax Kellermann2010-09-072-2/+10
| | | | False if the vector was not modified.
* output/httpd: implement "pause"Max Kellermann2010-08-311-0/+19
| | | | | Send silence to all connected clients while paused, to avoid connection interruption.
* output/httpd: forced flush after 32 kB of input dataMax Kellermann2010-08-312-0/+23
| | | | | Avoid buffer underruns on the streaming client, if the encoder is "too efficient" (e.g. when encoding silence while paused).
* ReplayGain filter: allow gain > 100 %Johan Kiviniemi2010-08-231-3/+11
| | | | | | | The ReplayGain filter clamped the gain to max. 100 % even if the algorithm determined the signal needed a boost. That would result in any such tracks being played with too low volume, effectively defeating the purpose of the filter.
* fd_util: add function dup_cloexec()Max Kellermann2010-08-032-0/+17
| | | | | | Unfortunately, there's no "optimized" implementation here. We can't use Linux's proprietary system call dup3(), because it would require us to specify the new descriptor.
* fd_util: add function recvmsg_cloexec()Max Kellermann2010-08-032-0/+49
|
* fd_util: add function socketpair_cloexec()Max Kellermann2010-08-032-0/+35
|
* playlist_song: fix memory leakMax Kellermann2010-07-251-0/+2
| | | | Free the temporary path string in apply_song_metadata().
* directory: free playlist vector in directory_free()Max Kellermann2010-07-251-0/+2
|
* queue_save: save tags and range of non-database songsMax Kellermann2010-07-255-21/+63
| | | | | Use the functions song_save() and song_load() to use the same format as in the database file for those songs which need the tags.
* song_save: save start_ms and end_msMax Kellermann2010-07-251-0/+11
| | | | | While this is not useful for the database, it may become useful for reusing this code for the state file.
* queue_save: simplify get_song()Max Kellermann2010-07-251-10/+3
| | | | Don't try db_get_song() if the URI has a scheme.
* {queue,song}_print: print relative paths if possibleMax Kellermann2010-07-254-1/+22
| | | | | | If a song with an absolute path points inside the music directory, print only the relative part. This happens when partial songs from a playlist file were loaded.
* queue_print: hide HTTP password in the "playlist" commandMax Kellermann2010-07-251-5/+2
| | | | | | | I've already changed the "playlistinfo" command to hide HTTP passwords, but forgot to do the same for the simpler "playlist" command. This patch changes queue_print_uris() to use the code from song_print_uri().
* queue_save: move code to queue_save_song()Max Kellermann2010-07-251-7/+11
|
* queue_save: queue_load_song() returns voidMax Kellermann2010-07-253-11/+7
| | | | | The only caller doesn't use its return value, and the value isn't useful anyway.
* song_save: song_load() returns NULL instead of falseMax Kellermann2010-07-251-2/+2
|
* state_file: use the text_file libraryMax Kellermann2010-07-253-38/+42
| | | | Don't use a large stack buffer.
* playlist_state: simplify printf() callsMax Kellermann2010-07-251-22/+18
| | | | Let the C compiler concatenate string constants.
* database, ...: remove EINTR checks after stdio callsMax Kellermann2010-07-255-18/+18
| | | | | | MPD doesn't have child processes anymore, and thus we're not expecting to receive SIGCHLD very often. Since hard disk access isn't interrupted by signals anyway, we don't need those excessive checks.
* directory_save: duplicate the playlist nameMax Kellermann2010-07-251-2/+9
| | | | | | The function playlist_metadata_load() will overwrite the input buffer before using the "name" parameter; since "name" points to the same buffer, we'll get a corrupted string.
* Added album tag support for libgme decoderRasi2010-07-231-0/+2
|
* update: store playlist files in databaseMax Kellermann2010-07-219-31/+353
| | | | | Don't open the music directory for each "lsinfo" call. Get the list of playlist files from the memory database.
* song_save: pass const songvec to songvec_save()Max Kellermann2010-07-212-2/+3
|
* song_save: include cleanupMax Kellermann2010-07-211-1/+0
|
* database: remove outdated commentMax Kellermann2010-07-211-1/+0
|
* tag_rva2: move code to rva2_apply_data()Max Kellermann2010-07-201-11/+16
|
* tag_rva2: add struct rva2_dataMax Kellermann2010-07-201-18/+39
| | | | | .. and several accessor inline functions, to simplify the calling code.
* tag_rva2: name the CHANNEL_* enumMax Kellermann2010-07-201-12/+12
|
* decoder/mad: move RVA2 code to tag_rva2.cMax Kellermann2010-07-203-89/+151
|
* Merge release 0.15.12 from branch 'v0.15.x'Max Kellermann2010-07-203-13/+31
|\ | | | | | | | | | | Conflicts: NEWS configure.ac
| * input/curl: remove assertion after curl_multi_fdset()Max Kellermann2010-07-201-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some users reported that MPD crashes when using a new CURL version with the threaded DNS resolver enabled. It seems that curl_multi_fdset() returns no file descriptor when the DNS resolver runs in another thread, so MPD does not have any event to wait for. On the CURL mailing list, somebody suggested to sleep for a fixed amount of time. This is not an elegant solution, because daemons should never have to sleep without waiting for an event. I hope the CURL developers will review the API and remove the threaded DNS resolver. Meanwhile, I'm removing the assertion in question, to allow those unfortunate users running the latest CURL version to continue using MPD.
| * input/curl: query timeout from CURLMax Kellermann2010-07-201-1/+18
| | | | | | | | | | Use curl_multi_timeout() to determine the select() timeout, instead of hard-coding one second.
| * tag_rva2: set "gain", not "peak"Max Kellermann2010-07-201-2/+2
| | | | | | | | RVA2 tags only store the "gain" value, there is no "peak" attribute.
| * decoder/mad: parse_rva2() returns boolMax Kellermann2010-07-201-9/+8
| |