aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* playlist: CamelCaseIsBadCourtney Cavin2009-07-2811-107/+128
| | | | Renamed all playlist functions to non-CamelCase.
* input/lastfm: Ensure multiple identical xml entities are decoded.Courtney Cavin2009-07-281-6/+5
| | | | | Previously, if two identical entities appeared in one string, only the first would get decoded. This fixes that bug.
* client: splitted client.c into several piecesMax Kellermann2009-07-2811-875/+1129
| | | | | The soure file client.c has nearly 1000 lines, time for splitting it into smaller pieces to improve readability.
* client: moved struct client to client_internal.hMax Kellermann2009-07-282-44/+69
| | | | Prepare splitting client.c into several sources.
* input/lastfm: use metadataCourtney Cavin2009-07-281-43/+297
| | | | | | Added a patch to flush out the last.fm input plugin slightly. It basically turns it into a wrapper for the appropriate plugin. Most notably metadata is now extracted.
* cmdline: obey $(sysconfdir) for default mpd.conf locationMax Kellermann2009-07-281-1/+0
| | | | | | | | | | | Instead of hard-coding the path "/etc/mpd.conf", use the configured $(sysconfdir) path. This can be set with: ./configure --sysconfdir=/etc Note that this changes the default path to "/usr/local/etc/mpd.conf", given the default prefix "/usr/local". This is actually more correct than the old default.
* client: moved some code to client_list_X()Max Kellermann2009-07-231-13/+51
| | | | Make the client list management a separate sub-library.
* client: don't include socket headersMax Kellermann2009-07-231-10/+0
| | | | | The client code uses portable GLib I/O functions and doesn't need the OS specific socket headers.
* player_thread: don't use precalculated size_to_timeMax Kellermann2009-07-232-22/+4
| | | | | | | Calculate the total play time with the audio_format object each time, using audio_format_time_to_size(). The function audioFormatSizeToTime() is not needed anymore, and will be removed with this patch.
* player_thread: moved code to update_song_tag()Max Kellermann2009-07-231-19/+27
|
* pcm_convert: use GError for error handlingMax Kellermann2009-07-2310-115/+177
| | | | | Don't abort the whole MPD process when the conversion fails. This has been a denial-of-service attack vector for years.
* pcm_resample_fallback: don't include glib.hMax Kellermann2009-07-221-1/+0
| | | | This library does not use GLib directly.
* pcm_resample_fallback: removed G_GNUC_UNUSED attributeMax Kellermann2009-07-222-6/+3
|
* pcm_channels: num_channels is unsignedMax Kellermann2009-07-222-12/+12
| | | | | You cannot have a negative number of channels, let's pass it as uint8_t instead of int8_t.
* pcm_byteswap: converted NULL checks to assertionsMax Kellermann2009-07-222-10/+5
| | | | | It is illegal to pass a NULL buffer to pcm_byteswap_X(). The result of this is that pcm_byteswap_X() never returns NULL.
* pcm_convert: added pcm_convert_state.byteswap_bufferMax Kellermann2009-07-222-3/+8
| | | | | | Currently, byteswapping is performed on the format_buffer. This can go wrong when this buffer is used twice during one run. Add a separate buffer for swapping the byte order.
* filter/volume: check the flag audio_format.reverse_endianMax Kellermann2009-07-221-0/+7
| | | | The volume plugin does not work for reverse_endian samples.
* audio_format: added API documentationMax Kellermann2009-07-221-0/+48
|
* audio_format: initialize reverse_endian in audio_format_init()Max Kellermann2009-07-221-0/+1
| | | | | This line was missing in the reverse_endian patch, and led to undefined values and crashes in that attribute.
* daemon: daemonize_close_stdin() optimised.Michal Nazarewicz2009-07-222-12/+7
| | | | | | | | | | | | | | | | | | Changed function to first close standard input (this may fail but we don't care) and then try to open /dev/null (this may fail but it shouldn't on Unix platforms plus we don't know what to do in such case anyways). Since standard input has the "zeroth" descriptor number next "open" will use it. Since there is no "/dev/null" on Windows (It's not even a valid path!) the second step is skipped if WIN32 is defined. As a final touch, since the function consists of merely two function calls it has been moved to header file and declared static inline. [mk: un-inline daemonize_close_stdin()]
* cmdline: renamed options.stderr to options.log_stderrEugeny N Dzhurinsky2009-07-223-6/+6
| | | | | | | On FreeBSD, "stderr" is a macro, and using this name for a struct member breaks the build. [mk: renamed _stderr to log_stderr]
* Merge branch 'master' of git://git.infradead.org/users/dwmw2/mpdMax Kellermann2009-07-1921-60/+240
|\ | | | | | | | | | | Conflicts: Makefile.am
| * Support wrong-endian ALSA outputDavid Woodhouse2009-07-191-2/+50
| |
| * Add reverse_endian field to struct audio_format and handle conversionDavid Woodhouse2009-07-196-5/+151
| |
| * Add audio_format_init() functionDavid Woodhouse2009-07-1915-53/+39
| | | | | | | | | | It makes no difference right now, but we're about to add an endianness flag and will want to make sure it's correctly initialised every time.
* | tag_ape: simplified the apeItems arrayMax Kellermann2009-07-191-22/+12
| | | | | | | | | | Make "enum tag_type" the array index, and convert apeItems to a sparse array.
* | tag_ape: moved code to tag_ape_import_item()Max Kellermann2009-07-191-12/+22
| | | | | | | | Improve code readability.
* | tag_ape: converted apeItems and tagItems to global varsMax Kellermann2009-07-191-20/+20
| | | | | | | | Don't initialize those arrays each time tag_ape_load() is called.
* | client: return "enum command_return" instead of "int"Max Kellermann2009-07-191-16/+24
| | | | | | | | | | Several functions work with the wrong return type, this patch fixes them.
* | removed buffer2array.cMax Kellermann2009-07-192-164/+0
| |
* | command: use the tokenizer libraryMax Kellermann2009-07-191-4/+50
| |
* | conf: use the tokenizer libraryMax Kellermann2009-07-191-62/+82
| |
* | tokenizer: new library replacing buffer2array()Max Kellermann2009-07-192-0/+235
| | | | | | | | | | The new code is more robust and more flexible. It provides detailed error information in GError objects.
* | daemon: Moved empty Windows version functions to header fileMichal Nazarewicz2009-07-192-21/+36
| | | | | | | | | | | | | | On Windows only daemonize_close_stdin() function does something. Other functions are either empty or generate an error. Those have been moved to header file and declared static inline so compiler can remove the call all together.
* | conf: replaced gcc "const" attribute with "pure"Michal Nazarewicz2009-07-191-13/+22
| | | | | | | | | | | | | | | | | | | | | | | | The functions were not "const" (they examined values pointed by arguments passed to them, quoting gcc's doc: "Note that a function that has pointer arguments and examines the data pointed to must _not_ be declared 'const'.") but rather "pure" and still not all of them. Note also, that even some of the functions declared "pure" are not pure, however, due to reasons stated in source code the attribute has been kept.
* | daemon: added "group" configuration optionMichal Nazarewicz2009-07-195-18/+35
|/ | | | | | | | | | | | | The "group" configuration option is similar to "user" as it sets user set what group MPD shall run as. With "user" option, MPD changed GID to the GID of the user, however, more control could be desired. Moreover, the patch changes the way of checking whether no setuid(2)/setgid(2) is required -- previously user names were compered, now UID and GIDs are compered (ie. the one we already have (getuid(2)/getgid(2)) with the one we want to change to).
* Merged release 0.15.1 from branch 'v0.15.x'Max Kellermann2009-07-166-19/+14
|\
| * output/httpd: removed duplicate sys/types.h includeMax Kellermann2009-07-141-4/+0
| | | | | | | | | | The first patch by Patrick didn't work, because his "#ifdef HAVE_OSX" line would have required config.h.
| * decoder/flac: fix assertion failure in tag_free() callMax Kellermann2009-07-061-2/+1
| | | | | | | | | | | | Initialize flac_data.tag right after flac_data_init(). This way, the "goto fail" won't jump to the point where tag_free(NULL) can be called.
| * output/httpd: include sys/types.hMax Kellermann2009-07-061-0/+1
| | | | | | | | | | | | On Mac OS X, the httpd plugin cannot be compiled, because OS X's system headers do nto include sys/types.h, although they use u_int32_t.
| * song: initialize mtime in song_alloc()Max Kellermann2009-07-061-0/+1
| |
| * log: fix double free() bug during shutdownMax Kellermann2009-07-051-6/+2
| | | | | | | | | | Don't free an internal configuration value in log_init(). Call config_get_path() instead of manually calling parsePath().
| * database: fixed NULL pointer dereference after charset changeMax Kellermann2009-06-301-4/+5
| | | | | | | | | | | | | | When the filesystem_charset is changed in mpd.conf, MPD should discard the old database. In this error branch, MPD did not fill the GError object properly, and logged a warning message instead, which caused a segmentation fault.
| * output_thread: don't play next chunk after command==PAUSEMax Kellermann2009-06-291-1/+5
| | | | | | | | | | When the PAUSE loop ends, re-check the next command before calling ao_play() again.
| * output_all: don't resume playback when stopping during pauseMax Kellermann2009-06-291-2/+0
| | | | | | | | | | | | | | | | When MPD was paused, and the client sent the "stop" command (or "clear"), a glitch caused MPD to continue playback for a split second. This was because audio_output_all_cancel() calls audio_output_all_update(), which reopens all output devices, and re-ignites the playback loop.
* | mapper: use g_file_test() instead of stat()Max Kellermann2009-07-151-20/+2
| | | | | | | | The GLib functions are more portable.
* | mapper: pass music and playlist directory to mapper_init()Max Kellermann2009-07-153-19/+23
| | | | | | | | Added another glue function in main().
* | main: moved complex initialization code to glue functionsMax Kellermann2009-07-151-17/+36
| | | | | | | | | | | | The glue_*() functions act as a glue between MPD's main() function and its libraries. They handle disabled features, and pass validated configuration options.
* | main: renamed openDB() to glue_db_init_and_load()Max Kellermann2009-07-151-2/+2
| | | | | | | | No CamelCase.
* | cmdline: renamed option "--stdout" to "--stderr"Max Kellermann2009-07-151-0/+2
| | | | | | | | | | | | Since version 0.14, MPD has been logging to standard error instead of standard output. The option name should reflect that. The old option continues to work, we will remove it in a future MPD release.