aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* client: eliminate variable "left" in client_write()Max Kellermann2008-10-171-5/+5
| | | | Reduce two temporary variables to only one.
* client: fixed send bufferMax Kellermann2008-10-171-49/+4
| | | | | There is no sense in using the kernel's send buffer size (SO_SNDBUF) for MPD's send buffer. Convert it into a static buffer of 4 kB.
* client: removed CLIENT_MAX_BUFFER_LENGTHMax Kellermann2008-10-171-3/+2
| | | | | | | Use a literal in the struct declaration, and sizeof(client->buffer) everywhere else. Also shrink the buffer from 40 kB to 4 kB. The buffer must only be large enough to hold one line of input, and 4 kB is still more than enough.
* client: added assertions on the buffer pointersMax Kellermann2008-10-171-0/+6
| | | | The buffer pointers must not exceed the buffer size.
* client: read() return value is ssize_tMax Kellermann2008-10-171-2/+2
| | | | Use ssize_t instead of int.
* client: handle partial lines correctlyMax Kellermann2008-10-171-2/+2
| | | | | | Commit 6eb62e47 didn't obey partial lines correctly: when a line wasn't finished in one read, the first part was ignored when the rest arrived.
* ffmpeg: new decoder pluginViliam Mateicka2008-10-173-0/+426
| | | | | | [mk: fixed indent, changed copyright statement, added autoconf test, fixed includes paths, fixed 2 gcc warnings, don't close input stream twice]
* Makefile.am: don't compile disabled decoder pluginsMax Kellermann2008-10-1716-159/+83
| | | | | Don't compile the sources of disabled decoder plugins at all, and don't attempt to register these.
* Makefile.am: don't compile disabled sourcesMax Kellermann2008-10-174-28/+21
| | | | | If a feature is disabled, don't compile the source file at all, disable it completely in Makefile.am instead.
* input_stream: removed nmemb argumentMax Kellermann2008-10-178-16/+12
| | | | | The nmemb argument isn't actually useful, and one of nmemb and size was always passed as 1. Remove it.
* input: declare struct input_streamMax Kellermann2008-10-177-43/+50
| | | | | Provide a struct type which can be forward-declared. The typedef InputStream is deprecated now.
* decoder: notify player after entering decodeStart()Max Kellermann2008-10-171-0/+1
| | | | | Wake up the player as soon as the decoder thread has entered its loop. This fixes a dead lock when the input is blocking.
* command: expect "file:///" url for local filesMax Kellermann2008-10-171-4/+7
| | | | | | When adding a local file, clients have to use the "file" URI schema described in RFC 1738 3.10. By adding this schema to "urlhandlers", a client can detect whether this feature is available.
* listen: fixed unused variable warning without HAVE_UCREDMax Kellermann2008-10-171-0/+2
| | | | The local variable "passcred" was only used by ucred code.
* configure.ac: check if "struct ucred" is availableMax Kellermann2008-10-161-2/+2
| | | | | | | | | | | By default, glibc 2.8 hides struct ucred behind the _GNU_SOURCE macro. I don't want to enable that globally, because it may encourage the use of non-portable functions. Test if "struct ucred" is available, and enable _GNU_SOURCE if required. For details about that issue, see glib's bug database: http://sources.redhat.com/bugzilla/show_bug.cgi?id=6545
* configure.ac: find more libraries with pkg-configMax Kellermann2008-10-161-0/+6
| | | | | Detect the following libraries with pkg-config: libshout, libid3tag, libmad.
* command: special case for "add /"Max Kellermann2008-10-161-1/+1
| | | | | The undocumented command "add /" adds the full music database to the playlist. Don't interpret this special path as a local file path.
* playlist: also allow world-readable local filesMax Kellermann2008-10-151-1/+1
| | | | | Allow a local user to not only add his own files, but also all world-readable files (mode 0444).
* song: check file type in song_file_update()Max Kellermann2008-10-151-1/+1
| | | | Don't load non-regular files.
* playlist: added support for adding songs not in the music databaseMax Kellermann2008-10-156-6/+59
| | | | | Clients which have authenticated via unix socket may add local files to the MPD playlist, provided that they own the file.
* command: print error message on "addid" failureMax Kellermann2008-10-151-1/+1
| | | | | | Returning the playlist_result value from a command handler does not make sense. Call print_playlist_result() there, and forward its return value.
* playlist: moved code to song_by_url()Max Kellermann2008-10-151-4/+17
| | | | | Replace some complicated checks from addToPlaylist() to the simpler function song_by_url().
* song: added song_in_database()Max Kellermann2008-10-152-9/+14
| | | | | | | Some functions assume that a song is not in the database when it is a remote song. Based on that, they decide whether they are responsible for freeing the song struct. Add a special function which checks whether a song is in the database (currently equal to song_is_file()).
* listen, client: enable SO_PASSCRED, get client's uidMax Kellermann2008-10-153-3/+41
| | | | | Enable authentication over unix sockets. Store the client's uid in the client struct.
* update: don't skip hidden filesMax Kellermann2008-10-151-3/+5
| | | | | Skip only the special directory entries "." and "..", don't skip all other "hidden" files.
* shout: check for vorbisenc libraryMax Kellermann2008-10-151-2/+2
| | | | | The switch from ogg.m4 to pkg-config intentionally disabled libvorbisenc. Enable it when shout_ogg is used.
* fixing several imports to work via pkg-configEnrico Weigelt2008-10-151-3/+10
| | | | | This patch fixes several imports to use pkg-config instead of certain esoteric tests.
* locate: use g_utf8_casefold() instead of string_toupper()Max Kellermann2008-10-155-30/+16
| | | | | | string_toupper() and strDupToUpper() were not able to deal with character sets other than US-ASCII. Use GLib's g_utf8_casefold() for strings.
* mapper: moved musicDir initialization from path.cMax Kellermann2008-10-155-31/+52
| | | | | Moved the musicDir variable and its initialization code from path.c to mapper.c.
* path: allow starting MPD with non-existing music directoryMax Kellermann2008-10-151-6/+6
| | | | When the music directory is not mounted yet, let MPD start anyway.
* path, tag_id3: use g_convert() instead of charConv.cMax Kellermann2008-10-155-234/+40
| | | | | | | 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.
* use GLibMax Kellermann2008-10-151-1/+4
| | | | | | 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?
* tag_id3: fix indentationMax Kellermann2008-10-151-28/+28
| | | | Indentation was broken in tag_id3.c: it used 4 spaces instead of tabs.
* idle: client command "noidle" aborts idleMax Kellermann2008-10-151-3/+21
|
* client: optimized client_input_received()Max Kellermann2008-10-151-38/+44
| | | | | | 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.
* listen: pass protocol family to establishListen()Max Kellermann2008-10-151-27/+8
| | | | | | 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.
* listen: use getaddrinfo() instead of gethostbyname()Max Kellermann2008-10-151-31/+22
| | | | | getaddrinfo() is more robust and has proper IPv6 support. The new code tries to bind to all IP addresses returned by getaddrinfo().
* command: added command "idle"Max Kellermann2008-10-1413-3/+284
| | | | | | "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.
* alsa: added #ifdefs around SND_PCM_NO_AUTO_xxxMax Kellermann2008-10-141-0/+6
| | | | | | 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.
* playlist: don't use uninitialized local variable (typo)Max Kellermann2008-10-141-1/+1
| | | | | Instead of the uninitialized local variable "s", I should have used "uri".
* oss: use unsigned integer for ioctl constantsMax Kellermann2008-10-141-12/+12
| | | | The OSS constants overflow a signed integer, use unsigned instead.
* oss: convert several macros to enumMax Kellermann2008-10-141-19/+25
| | | | C enums are nicer than CPP macros.
* alsa: optionally disable resampling and othersMax Kellermann2008-10-141-1/+15
| | | | | | 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.
* changed package name to "mpd-mk", updated version numberMax Kellermann2008-10-141-1/+2
| | | | | | 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.
* update: don't follow relative symlinksMax Kellermann2008-10-141-1/+44
| | | | | | 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.
* ls: removed myStat(), isFile(), isDir()Max Kellermann2008-10-142-53/+0
| | | | The mapper code has replaced these functions.
* mapper: new song-to-filesystem mapper libraryMax Kellermann2008-10-149-53/+241
| | | | | | 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: moved code to playlist_save.cMax Kellermann2008-10-145-26/+89
| | | | | playlist_print_song() and playlist_print_uri() handle charset conversion and (optional) music directory prefixing.
* song: pass const song pointer to song_get_url()Max Kellermann2008-10-142-2/+2
| | | | song_get_url() doesn't modify the song object.
* {dir,song}vec: fix off-by-one errors in {dir,song}vec_deleteEric Wong2008-10-142-10/+9
| | | | | | | Found by Valgrind while looking for another bug... Hmm.. I should really just make this code generic since they're duplicated...