aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
| | * Insure proper initialization of stack-allocated struct.Christopher Brannon2011-02-151-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Version 1.0.0 of the libao library added a new field to the ao_sample_format struct. It is a char * named matrix. When an ao_sample_format is allocated on the stack, this field contains garbage. The proper course is to insure that is initialized to NULL. NULL indicates that we do not want any mapping. The struct is now initialized using a static initializer, and this technique is compatible with all known versions of libao.
| | * update: log all file permission problemsMax Kellermann2011-01-311-0/+6
| | |
| | * Fix NDEBUG testAndreas Wiese2011-01-141-1/+1
| | | | | | | | | | | | | | | | | | | | | <stdbool.h> needs to be included unconditionally from definition of NDEBUG, since »bool« doesn't get defined otherwise. Signed-off-by: Andreas Wiese <aw-devel@meterriblecrew.net>
| * | output/httpd: include sys/socket.h for AF_UNIXUlrich Spörlein2011-03-091-0/+1
| | |
| * | output/oss: disable 24 bit playback on FreeBSDMax Kellermann2011-02-281-0/+9
| | | | | | | | | | | | See code comment.
| * | output/oss: AFMT_S24_PACKED is little-endianMax Kellermann2011-02-281-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | According to the Solaris dsp manpage, AFMT_S24_PACKED is little-endian: http://download.oracle.com/docs/cd/E19963-01/821-1475/6nmf5baot/index.html The Minix soundcard.h header says the same.
| * | general: whitespace cleanupThomas Jansen2011-02-099-29/+29
| | | | | | | | | | | | | | | Remove trailing whitespace found by this command: find -name '*.[ch]' | xargs grep "[[:space:]]$"
| * | output/httpd: initialize unflushed_inputThomas Jansen2011-02-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following valgrind warning occuring on the first call of httpd_output_read_page: ==20124== Conditional jump or move depends on uninitialised value(s) ==20124== at 0x425E65: httpd_output_read_page (httpd_output_plugin.c:240) ==20124== by 0x426087: httpd_output_open (httpd_output_plugin.c:279) ==20124== by 0x41D862: ao_open (output_plugin.h:206) ==20124== by 0x41E133: audio_output_task (output_thread.c:590)
| * | Set fadeout in gme_decoder_plugin. Due to the nature of the gme library,Tony Miller2011-02-031-0/+3
| | | | | | | | | | | | this needs to be done for the end of songs to be detected.
* | | RoarAudio output pluginHans-Kristian Arntzen2011-03-165-0/+512
| | |
* | | command: "update" and "rescan" need only "CONTROL" permissionMax Kellermann2011-02-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From http://bugs.debian.org/513291 "In mpd.conf, the "admin" permission covers updating the db and killing mpd. "Since there are quite some usecases in which the user can upload music to the mpd's directory by means of anonymous FTP or so, it is desirable that any user may issue a db update, while killing the mpd should not be possible. "I'd suggest to remove the db update from the admin group and either add it to "control" or an own group."
* | | playlist_state: add option "restore_paused"Max Kellermann2011-02-232-0/+9
| | | | | | | | | | | | | | | When set, MPD will not auto-start playback on startup; it will be in "paused" state.
* | | playlist_state: declare local variable as enumMax Kellermann2011-02-231-1/+1
| | |
* | | output/shout: add possibility to set urlThomas Jansen2011-02-091-0/+7
| | | | | | | | | | | | Added a new optional parameter for the shout plugin called "url".
* | | protocol: support client-to-client communicationMax Kellermann2011-01-299-0/+562
| | |
* | | client_idle: export client_idle_add()Max Kellermann2011-01-292-5/+14
| | |
* | | client_idle: add header client_idle.hMax Kellermann2011-01-295-13/+45
| | |
* | | copyright year 2011Max Kellermann2011-01-29428-429/+429
| | |
* | | input/curl: enable CURLOPT_NETRCIan Zimmerman2011-01-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I wanted mpd to play a mp3 stream from a music website. The stream is only available to subscribers, which restriction is enforced through normal http authentication. However, the URL I get from the website is not the final URL of the stream, but a generic URL which points to the real one through a redirect (code 301). Thus, I cannot predict the final URL, and so I cannot use the username:password hack to force the authentication, and mpd (libcurl on mpds behalf) fails to grab the stream. libcurl allows the option CURLOPT_NETRC to be set and then the credentials can be stored in the good old .netrc file (in this case it would be ~mpd/.netrc, of course). But mpd doesn't set this option. I think it should.
* | | output/raop: new output pluginDustin Puckett2011-01-285-0/+1669
| | | | | | | | | | | | Remote Audio Output Protocol (RAOP), for Apple devices.
* | | mpg123 decoder: implement seekingYuriy Kaminskiy2011-01-211-2/+14
| | |
* | | mpg123 decoder: report bitrateYuriy Kaminskiy2011-01-211-1/+25
| | |
* | | decoder_thread: fix assertion failure at song endMax Kellermann2011-01-161-2/+1
| | | | | | | | | | | | | | | | | | Don't finish the current command twice. This bug was never noticed, but was revealed by a new assertion check.
* | | output_control: document internal functionsMax Kellermann2011-01-101-0/+17
| | |
* | | output_control: move code to ao_lock_command()Max Kellermann2011-01-101-9/+15
| | |
* | | output_control: ao_command() calls ao_command_async()Max Kellermann2011-01-101-7/+6
| | | | | | | | | | | | Merge some code.
* | | update_remove: use plain GCond instead of notifyMax Kellermann2011-01-101-9/+16
| | | | | | | | | | | | GMutex/GCond guarantee that the access to removed_song is protected.
* | | include cleanupMax Kellermann2011-01-103-1/+4
| | |
* | | decoder_control: store GCond object, not a player_controlMax Kellermann2011-01-107-28/+20
| | | | | | | | | | | | | | | | | | Remove the decoder dependency on player_control. All player_control was needed for is to signal the player thread, and we can do that with a simple GCond as well.
* | | decoder_thread: move code to decoder_command_finished_locked()Max Kellermann2011-01-101-8/+19
| | |
* | | decoder_control: remove unused function dc_command_wait()Max Kellermann2011-01-102-11/+0
| | | | | | | | | | | | Only dc_command_wait_locked() is really being used.
* | | decoder_control: replace dc_init() with dc_new()Max Kellermann2011-01-103-15/+17
| | | | | | | | | | | | | | | dc_new() allocates the object and returns it. dc_free() frees it (replaces dc_deinit()).
* | | player_control: removed the global variable "pc"Max Kellermann2011-01-1041-571/+738
| | | | | | | | | | | | | | | | | | | | | Allocate a player_control object where needed, and pass it around. Each "client" object is associated with a "player_control" instance. This prepares multi-player support.
* | | dbUtils: don't use directoryAddSongToPlaylist() twiceMax Kellermann2011-01-101-1/+1
| | | | | | | | | | | | | | | In findAddInDirectory(), call playlist_append_song() directly, to have some more type checking.
* | | input_init: add assertions on completeness of pluginsMax Kellermann2011-01-081-0/+6
| | |
* | | input/cdio_paranoia: add module nameMax Kellermann2011-01-081-0/+1
| | | | | | | | | | | | | | | Seems like we forgot to give this module a name... this can crash MPD on startup.
* | | Merge branch 'v0.16.x'Max Kellermann2011-01-0710-57/+65
|\| | | | | | | | | | | | | | | | | Conflicts: NEWS configure.ac
| * | Merge branch 'v0.15.x' into v0.16.xMax Kellermann2011-01-073-2/+12
| |\| | | | | | | | | | | | | | | | | | | Conflicts: NEWS configure.ac src/directory.h
| | * encoder/lame: explicitly configure the output sample rateMax Kellermann2011-01-071-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | When you don't explicitly set an output sample rate, liblame tries to guess an output sample rate from the input sample rate. You would think that this "guessing" consists of just setting both equal, but that is not the case. For 44.1kHz at 96kbit/s, liblame chooses 32kHz. This patch explicitly configures the output sample rate, to stop the bad guessing.
| | * output/httpd: define G_LOG_DOMAIN in httpd_client.cMax Kellermann2011-01-071-0/+3
| | |
| | * directory: fix warning "comparison between signed and unsigned"Max Kellermann2010-12-211-2/+2
| | | | | | | | | | | | Cast the constant to dev_t, not to unsigned.
| | * zeroconf-bonjour: use g_htons() instead of htons()Max Kellermann2010-12-211-1/+1
| | | | | | | | | | | | Fixes the gcc warning "implicit declaration of function 'htons'".
| * | player_thread: discard empty chunks while cross-fadingMax Kellermann2011-01-071-0/+13
| | | | | | | | | | | | | | | | | | When a music_chunk to be crossfaded consists only of a tag, cross-fading is not possible, and led to an assertion failure. This patch just discards those, as if cross-fading was not enabled.
| * | output_thread: fix double lockMax Kellermann2011-01-071-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | During the whole output thread, the audio_output object is locked, and it is only unlocked while waiting for the GCond and while running a plugin method. The error handler in ao_play_chunk() attempted to lock the object again, which was code from MPD 0.15.x which should have been removed a long time ago.
| * | player_thread: fix assertion failure due to early seekMax Kellermann2011-01-071-0/+4
| | | | | | | | | | | | | | | | | | | | | Until the decoder plugin has called decoder_initialized(), the player may not submit seek commands. This however could occur with a slow decoder and a CUE file with a virtual song offset. This patch adds another check.
| * | player_thread: make variables more localMax Kellermann2011-01-071-36/+19
| | |
| * | output/osx: fix up audio format first, then apply it to deviceMax Kellermann2011-01-071-9/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | This is a MPD 0.16 regression: when playing a 24 bit file, the switch to 16 bit was made only partially, after mBytesPerPacket and mBytesPerFrame had already been applied. That means mBytesPerFrame referred to 24 bit, and mBitsPerChannel referred to 16 bits. Of course, that cannot work.
| * | add void casts to suppress "result unused" warnings (clang)Max Kellermann2010-12-212-3/+3
| | |
| * | decoder/mad: work around build failure on SolarisAlex Viskovatoff2010-12-211-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Rename the "version" struct, because it seems to be a reserved name on Solaris: "src/decoder/mad_decoder_plugin.c", line 550: (enum) tag redeclared: version cc: acomp failed for src/decoder/mad_decoder_plugin.c
| * | output/solaris: add missing parameter to open_cloexec() callAlex Viskovatoff2010-12-211-1/+1
| | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-09 06:53:52 +0000