aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* player_thread: fix assertion failure due to wrong music pipe on seekMax Kellermann2010-11-051-0/+8
| | | | | | | | | When one song is played twice, and the decoder is working on the second "instance", but the first should be seeked, the check in player_seek_decoder() may assume that it can reuse the decoder without exchanging pipes. The last thing was the mistake: the pipe pointer was different, which led to an assertion failure. This patch adds another check which exchanges the player pipe.
* decoder_control: pass music_pipe to dc_start()Max Kellermann2010-11-053-12/+18
| | | | More abstraction for decoder_control.pipe.
* player_thread: add helper function player_dc_at_next_song()Max Kellermann2010-11-051-6/+33
| | | | Some abstraction for decoder_control.pipe access.
* pipe: add helper function music_pipe_empty()Max Kellermann2010-11-053-4/+10
|
* input/rewind: remove redundant NULL check before g_free() callMax Kellermann2010-11-051-2/+1
|
* input/rewind: add two assertionsMax Kellermann2010-11-051-0/+3
|
* input/rewind: fix double free bugMax Kellermann2010-11-051-0/+1
| | | | | Duplicate the "mime" attribute of the inner input_stream object, instead of copying the pointer.
* decoder/{mp4ff,ffmpeg}: add extension ".m4b" (audio book)Max Kellermann2010-11-052-2/+10
| | | | Same as ".m4a".
* output_thread: fix assertion failure due to race condition in OPENMax Kellermann2010-11-041-1/+9
| | | | | | Change the assertion on "fail_timer==NULL" in OPEN to a runtime check. This assertion crashed when the output thread failed while the player thread was calling audio_output_open().
* output_internal: protect attribute "fail_timer" with mutexMax Kellermann2010-11-042-1/+7
|
* output_control: lock object in audio_output_open()Max Kellermann2010-11-041-6/+19
| | | | Protect the attributes "open" and "fail_timer".
* output_control: lock object in audio_output_close()Max Kellermann2010-11-041-2/+28
| | | | Protect the attributes "open" and "fail_timer".
* output/httpd: MIME type audio/ogg for Ogg VorbisMax Kellermann2010-10-031-1/+1
| | | | | | RFC 5334 10.3 defines the MIME type "audio/ogg". We could use "application/ogg" as well, but we know for sure that we only emit audio data.
* input/rewind: enable for MMSThomas Jansen2010-09-281-2/+10
|
* rewind_input_plugin: Update MIME not only onceThomas Jansen2010-09-231-3/+4
| | | | | | | The assumption that MIME type is set only once is not valid with CURL, as URL redirections may update the MIME type. This fixes bug #3044.
* Correctly terminate stream_title.Qball Cow2010-09-081-0/+1
| | | | | This caused random data to be send via icy-server if the played song had no tags.
* input/curl: fix version check for curl_multi_timeout()Max Kellermann2010-09-071-1/+1
| | | | | | | According to the CURL web site, curl_multi_timeout() was added in version 7.15.4: http://curl.haxx.se/libcurl/c/curl_multi_timeout.html
* output_thread: fix race condition after CANCEL commandMax Kellermann2010-08-191-0/+10
| | | | | | Clear the notification before finishing the CANCEL command, so the notify_wait() after that will always wait for the right notification, sent by audio_output_all_cancel().
* notify: add function notify_clear()Max Kellermann2010-08-192-0/+12
|
* 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
|
* decoder/wildmidi: support version 0.2.3Max Kellermann2010-07-201-0/+4
| | | | | | | In libwildmidi 0.2.3, the function WildMidi_SampledSeek() was removed, without changing the SO name. This patch adds an autoconf check for that function. Fall back to WildMidi_FastSeek() if WildMidi_SampledSeek() is not available anymore.
* decoder/ffmpeg: fix libavformat 0.6 by using av_open_input_stream()Max Kellermann2010-06-301-82/+52
| | | | | | | | | | | | libavformat 0.6 does not pass the original URI pointer to the "open" method, which leads to a crash because MPD was using a dirty hack to pass a pointer to that method. This patch switches to av_open_input_stream() with a custom ByteIOContext class, instead of doing the URI string hack with av_open_input_file(). Loosely based on a patch from Jasper St. Pierre.
* decoder/ffmpeg: manual format probingMax Kellermann2010-06-301-4/+47
| | | | | | | | | Use the libavformat function av_probe_input_format() to probe the AVInputFormat, instead of letting av_open_input_file() do it implicitly. We will switch to av_open_input_stream() very soon, which does not have the probing code. Loosely based on a patch from Jasper St. Pierre.
* decoder/ffmpeg: free URI, fix memory leakMax Kellermann2010-06-301-1/+3
| | | | Free the string allocated by decoder_get_uri().
* decoder/vorbis: handle uri==NULLMax Kellermann2010-06-301-0/+3
| | | | This fixes a theoretical crash, which has never occurred in practice.
* ffmpeg: read more metadata.Anton Khirnov2010-06-301-1/+10
|
* decoder/ffmpeg: free AVFormatContext on errorMax Kellermann2010-06-301-0/+4
| | | | Fix a memory leak in some code paths.
* pcm_buffer: make the buffer pointer "void"Max Kellermann2010-06-301-1/+1
|
* decoder/mp4ff: support tag "album artist"Max Kellermann2010-06-301-0/+1
| | | | | We already supported "albumartist", but it seems some folks also use "album artist" (with a space).
* Make get_remote_uid() work on BSDAndreas Vögele2010-06-301-1/+7
| | | | | | I've attached a patch that will make file URIs work on operating systems that provide the getpeereid() function call to check the user ID of the peer connected to a UNIX domain socket.
* playlist: emit IDLE_OPTIONS when resetting single modeMax Kellermann2010-06-301-0/+3
|
* directory_print: return voidMax Kellermann2010-06-302-7/+3
| | | | There is no useful return value here.
* decoder/mikmod: fix memory leakMax Kellermann2010-06-301-2/+4
| | | | | The return value of Player_LoadTitle() is allocated with malloc(), and must be freed by the caller.
* decoder/mp4ff: remove duplicate entries in the tag name tableMax Kellermann2010-06-301-9/+5
| | | | Reuse the function tag_name_parse_i().
* decoder/mp4ff: moved code to mp4ff_tag_name_parse()Max Kellermann2010-06-301-1/+7
|
* tag_ape: remove duplicate entries in the tag name tableMax Kellermann2010-06-301-7/+5
| | | | Reuse the function tag_name_parse_i().
* tag: added function tag_name_parse()Max Kellermann2010-06-303-12/+60
| | | | Convert a string into a tag_type enum.
* tag_ape: move code to tag_ape_name_parse()Max Kellermann2010-06-301-1/+7
|
* decoder/vorbis: use single global ov_callbacks constantMax Kellermann2010-06-301-7/+9
| | | | Initialize the ov_callbacks struct at compile time.
* decoder/mp4ff: support tags "albumartist", "band"Max Kellermann2010-06-301-0/+2
| | | | | I'm not sure if mapping "band" to TAG_PERFORMER is correct, but it might be better than nothing.
* decoder/mp4ff: use tag_table.h to parse tag namesMax Kellermann2010-06-301-18/+15
| | | | Convert if/else/else/... to a loop.
* tag_ape: move table lookup to tag_table.hMax Kellermann2010-06-302-9/+52
| | | | Allow code sharing.
* tag_ape: support album artistMax Kellermann2010-06-301-0/+1
| | | | | | | | | | I took this tag name from a MusePack sample file I got from a user. It is not documented in the APE specification: http://wiki.hydrogenaudio.org/index.php?title=APE_key People seem to be using undocumented extensions to the specification anyway, and the best we can do is attempt to support them.
* tag_ape: simplified the apeItems arrayMax Kellermann2010-06-301-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 Kellermann2010-06-301-12/+22
| | | | Improve code readability.
* tag_ape: converted apeItems and tagItems to global varsMax Kellermann2010-06-301-20/+20
| | | | Don't initialize those arrays each time tag_ape_load() is called.
* decoder/mad: fix buffer variable name on !HAVE_ID3TAGMax Kellermann2010-05-301-3/+3
|