| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
GLib provides an easier API for character set conversion than iconv().
Use g_convert() / g_convert_with_fallback() for all character
conversions. We should optimize the path.h API later to return a
newly allocated buffer, so we can just pass GLib's return value.
|
|
|
|
|
|
| |
GLib is a nice and portable utility library. We are going to use it
from now on, and eliminate a lot of duplicated code from MPD. Why
invent the wheel again and again?
|
|
|
|
| |
Indentation was broken in tag_id3.c: it used 4 spaces instead of tabs.
|
| |
|
|
|
|
|
|
| |
Use memchr() instead of manually traversing the input buffer. Update
the client's properties after all commands have been processed. Check
for buffer overflow once.
|
|
|
|
|
| |
The manual page suggests that you can specify multiple servers, but it
did not clarify what that means.
|
|
|
|
|
|
| |
The caller already knows the protocol family, and we can eliminate the
complicated switch statement in establishListen() if we just pass this
information. This seems more robust.
|
|
|
|
|
| |
getaddrinfo() is more robust and has proper IPv6 support. The new
code tries to bind to all IP addresses returned by getaddrinfo().
|
|
|
|
|
|
| |
"idle" waits until something noteworthy happens on the server,
e.g. song change, playlist modified, database updated. This allows
clients to keep up to date without polling.
|
|
|
|
|
|
| |
These macros are not available in older libasound versions (1.0.13
fails, 1.0.16 is ok). Ignore the configuration if the constants are
not defined.
|
|
|
|
|
| |
Makefile.am sets docdir = $(prefix)/share/doc/$(PACKAGE). This
overrides the setting specified in ./configure --docdir.
|
|
|
|
|
| |
Instead of the uninitialized local variable "s", I should have used
"uri".
|
| |
|
|
|
|
|
| |
Removed some configure checks which aren't used anymore since we use
C99.
|
|
|
|
|
| |
Why check for C++ and fortran compilers? This hack was borrowed from
Qball's gmpc git repository.
|
|
|
|
|
| |
Add more configure options which control how MPD is compiled.
--enable-debug is always enabled, until there is a stable release.
|
|
|
|
| |
Yet another gcc flag which gives us lots of interesting warnings.
|
|
|
|
| |
The OSS constants overflow a signed integer, use unsigned instead.
|
|
|
|
| |
C enums are nicer than CPP macros.
|
|
|
|
|
| |
Some warning flags may cause errors during library detection code. Do
the CFLAGS checks at the end.
|
|
|
|
|
|
| |
Added mpd.conf options for disabling automatic resamling, sample
format and channel conversion. This way, users may choose to override
ALSA's automatic resampling, and use libsamplerate instead.
|
|
|
|
| |
A ".TP" was missing, and broke the manual page's layout.
|
|
|
|
|
|
| |
This git branch has become a real MPD fork now. Time to change the
package name to the code name "mpd-mk". Set the version number to
"0.14~git" to mark this as a non-released version.
|
|
|
|
|
|
| |
Don't follow relative symlinks which point into the music directory.
This allows you to organize music with symbolic links, without MPD
managing separate copies of each song.
|
|
|
|
| |
The mapper code has replaced these functions.
|
|
|
|
|
|
| |
The mapper library maps directory and song objects to file system
paths. With this central library, the code mixture in path.c should
be cleaned up, and we will be able to add neat features like aliasing.
|
|
|
|
|
| |
playlist_print_song() and playlist_print_uri() handle charset
conversion and (optional) music directory prefixing.
|
|
|
|
| |
song_get_url() doesn't modify the song object.
|
|
|
|
|
|
|
| |
Found by Valgrind while looking for another bug...
Hmm.. I should really just make this code generic since
they're duplicated...
|
|
|
|
|
|
|
| |
isMusic() used to be a very inefficient function: with every
invocation, it did another stat() on the specified file. There is
only one caller, do the stat() there manually and use hasMusicSuffix()
instead of isMusic().
|
|
|
|
|
|
|
| |
By always creating the parent directory, we can use delete_name_in()
without further lookups. The parents which may non exist will be
pruned later. An update request for a non-existing or empty directory
should be quite unusual, so this doesn't add any measurable overhead.
|
|
|
|
|
|
| |
In order to optimize buffer usage, pass only the base file name to
updateInDirectory(). This way, updateInDirectory() may choose when to
allocate a larger buffer for the full path.
|
| |
|
|
|
|
|
| |
delete_name_in() is similar to delete_path(), but it does not need to
look up the parent directory.
|
|
|
|
|
|
| |
It is invalid to pass a path with the wrong dirname to dirvec_find().
To be able to find a subdirectory only by its basename, compare only
the basename of both paths.
|
|
|
|
| |
directory_get_name() returns the base name of the directory.
|
|
|
|
|
|
| |
The only caller of deletePlaylist() appends PLAYLIST_FILE_SUFFIX, so
we can be sure it's already there. We don't need to stat the file,
since unlink() does all the checking.
|
|
|
|
|
|
|
| |
Commit 80a2c937 broke resume after pause: it cleared the
input_audio_format when it attempted to simplify a complicated
expression. Don't clear it, just assign input_audio_format if a new
format was specified.
|
|
|
|
|
| |
The inline functions directory_is_empty() and directory_get_path()
don't modify the object - pass constant object pointers to them.
|
|
|
|
|
|
|
| |
We only need to lock sv->nr changes to prevent traversals ( why
it's called "nr_lock"). free(3) is a "slow" function on my
system; so we can avoid unnecessarily holding a lock long for
longer than needed.
|
|
|
|
|
|
| |
Previously only updates with subdirectories being
specified could be queued. No harm in queueing
full updates.
|
|
|
|
| |
This way we avoid unnecessary heap allocations.
|
|
|
|
|
| |
Not all compilers support struct packing, and those that don't
shouldn't be punished for it.
|
| |
|
|
|
|
|
| |
Added configuration parameter "protocol" which lets the user choose
from 3 shout protocols. This adds support for real shoutcast servers.
|
|
|
|
|
|
| |
Case insensitivity isn't helpful, and comparing only the first 3 bytes
of a configured value may encourage users to supply wrong or
misleading values.
|
|
|
|
|
|
| |
If the sample format isn't supported by the device (i.e. 24 bit on
low-end sound chips), fall back to 16 bit output. There is code in
pcm_utils.c which converts PCM data to 16 bit.
|
|
|
|
|
| |
When calculating the conversion buffer size, don't hard-code the
formulas for only mono<->stereo.
|
|
|
|
|
|
|
| |
Convert any number of channels to stereo. In fact, this isn't really
stereo, it's rater mono blown up to stereo. This patch should only
make it possible to play 5.1 files at all; "real" conversion to stereo
should be implemented, but for now, this is better than nothing.
|
|
|
|
|
|
| |
In order to be able to deal with non-trivial conversions,
pcm_convertChannels() needs to know both the input and the output
channel count. Simplify buffer allocation in that function.
|