| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
The g_free() function includes a NULL check. We don't have to do it
twice.
|
|
|
|
| |
This causes a segmentation fault...
|
|
|
|
|
| |
This way, we don't have to pass the full "Options" object to
daemonize().
|
|
|
|
|
|
| |
The constructor/destructor functions parse and free the configuration
properly. This way, we don't have to load the pid file path more than
once.
|
|
|
|
| |
The state_file library should manage its own regular saves.
|
|
|
|
|
| |
The constructor/destructor interface should hide the functions
read_state_file() and write_state_file().
|
| |
|
|
|
|
| |
config_get_path() is an simpler interface than parseConfigFilePath().
|
|
|
|
|
| |
Fetch the configuration variables buffered_chunks and
buffered_before_play just when they are needed.
|
|
|
|
|
| |
Moved changeToUser(), cleanUpPidFile(), killFromPidFile() to
daemon.c. These are daemonization functions.
|
|
|
|
| |
Without a music_directory, MPD is an excellent streaming client.
|
| |
|
|
|
|
| |
Don't include conf.h in database.c.
|
|
|
|
|
| |
time(NULL) shows the wrong results when the machine's clock is
changed.
|
|
|
|
|
| |
Don't use dbUtils.h functions. This reduces 4 full database walks to
just one.
|
|
|
|
|
| |
Don't recalculate the number of artists and albums each time a client
requests statistics. Calculate that once in stats_update().
|
|
|
|
| |
Renamed functions and types.
|
| |
|
|
|
|
| |
Don't return a writable pointer.
|
|
|
|
| |
No "force" parameter, pass a default value instead.
|
| |
|
|
|
|
| |
Renamed functions, types, variables.
|
|
|
|
|
|
| |
If a song is not within the music directory ("file:///..."), it has no
"parent directory". The archive code nonetheless dereferences the
parent pointer, causing a segmentation fault. Check parent!=NULL.
|
|
|
|
|
|
|
| |
One of the previous patches made MPD consume 100% CPU in a busy wait:
when the music_pipe was full, it did not wait (with notify_wait()) for
free chunks, because a variable has a different meaning now. Always
pass "true" as the "wait" parameter.
|
|
|
|
|
|
| |
Some plugins used the APE or ID3 tag loader as a fallback when their
own methods of loading tags did not work. Move this code out of all
decoder plugins, into song_file_update().
|
| |
|
|
|
|
|
|
| |
Removed yet another superfluous buffer layer: return the PCM buffer
from pcm_convert() instead of copying PCM data into the
caller-supplied buffer.
|
|
|
|
|
| |
The function is unused, since decoder_api() uses music_pipe_write() /
music_pipe_expand() now.
|
|
|
|
|
| |
Copy PCM data to the music_pipe_write() buffer, and apply replay gain
/ normalization to it, instead of manipulating the source buffer.
|
|
|
|
|
|
|
| |
This new API gives the caller a writable buffer to the music pipe
chunk. This may allow the caller to eliminate several buffer copies,
because it may manipulate the returned buffer, until it calls
music_pipe_expand().
|
|
|
|
|
|
|
|
|
|
|
|
| |
When libvorbis knows that a song is seekable, it seeks around like
crazy in the file before starting to decode it. This is very
expensive on remote HTTP resources, and delays MPD for 10 or 20
seconds.
This patch disables seeking on remote songs, because the advantages of
quickly playing a song seem to weigh more than the theoretical ability
of seeking for most MPD users. If users feel this feature is needed,
we will make a configuration option for that.
|
|
|
|
| |
Decoder plugins must not send partial frames.
|
|
|
|
|
| |
Due to rounding errors, it was possible that the fallback resampler
returned partial frames.
|
|
|
|
|
|
| |
getBoolConfigParam() returns an int. It is not possible to check for
CONF_BOOL_UNSET after it has been assigned to a bool; use a temporary
int value for that.
|
| |
|
|
|
|
| |
Older gcc versions complained about shadowed parameters in prototypes.
|
|
|
|
| |
It returns bool, not int.
|
|
|
|
|
|
| |
When MPD starts without audio output configuration, the "param"
variable is NULL. This triggers a segmentation fault in both mixer
plugins.
|
| |
|
|
|
|
| |
When tremor (libvorbisidec) is used, HAVE_OGGVORBIS was not defined.
|
|
|
|
|
| |
This patch allows mpd to recognise the albumartist tag in the way
foobar2000 and others write it to files.
|
|
|
|
|
| |
Splitted flac_copy_vorbis_comment() into flac_copy_comment() and
flac_copy_comment().
|
|
|
|
|
| |
Simplify flac_copy_vorbis_comment() by moving the comment
identification code out.
|
|
|
|
|
| |
Free the tag object when it turns out to be empty. This simplifies
several functions and APIs.
|
|
|
|
|
| |
tag_is_defined() checks whether there is any information in the tag
object.
|
| |
|
|
|
|
| |
Use tag_is_empty() instead.
|
|
|
|
| |
Renamed types, functions, variables.
|
|
|
|
|
|
| |
Calling input_curl_select() after EOF has been reached causes an
assertion failure. This can happen if the HTTP response is empty.
Check c->eof before calling input_curl_select().
|
|
|
|
| |
Set the "ready" flag for empty resources.
|