| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Avoid buffer underruns on the streaming client, if the encoder is "too
efficient" (e.g. when encoding silence while paused).
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
| |
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.
|
| |
|
| |
|
|
|
|
| |
Free the temporary path string in apply_song_metadata().
|
| |
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
While this is not useful for the database, it may become useful for
reusing this code for the state file.
|
|
|
|
| |
Don't try db_get_song() if the URI has a scheme.
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
| |
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().
|
| |
|
|
|
|
|
| |
The only caller doesn't use its return value, and the value isn't
useful anyway.
|
| |
|
|
|
|
| |
Don't use a large stack buffer.
|
|
|
|
| |
Let the C compiler concatenate string constants.
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
|
| |
Don't open the music directory for each "lsinfo" call. Get the list
of playlist files from the memory database.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
.. and several accessor inline functions, to simplify the calling
code.
|
| |
|
| |
|
|\
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| | |
Use curl_multi_timeout() to determine the select() timeout, instead of
hard-coding one second.
|
| |
| |
| |
| | |
RVA2 tags only store the "gain" value, there is no "peak" attribute.
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | |
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
src/decoder/ffmpeg_decoder_plugin.c
src/decoder/mikmod_decoder_plugin.c
src/decoder/mp4ff_decoder_plugin.c
src/decoder/vorbis_decoder_plugin.c
src/directory_print.c
src/playlist_control.c
src/tag_ape.c
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| | |
Free the string allocated by decoder_get_uri().
|
| |
| |
| |
| | |
This fixes a theoretical crash, which has never occurred in practice.
|
| | |
|
| |
| |
| |
| | |
Fix a memory leak in some code paths.
|
| | |
|
| |
| |
| |
| |
| | |
We already supported "albumartist", but it seems some folks also use
"album artist" (with a space).
|
| |
| |
| |
| |
| |
| | |
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.
|
| | |
|
| |
| |
| |
| | |
There is no useful return value here.
|
| |
| |
| |
| |
| | |
The return value of Player_LoadTitle() is allocated with malloc(), and
must be freed by the caller.
|
| |
| |
| |
| | |
Reuse the function tag_name_parse_i().
|
| | |
|