aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* stored_playlist: fix integer overflow in length estimationMax Kellermann2008-12-241-1/+1
| | | | | | | With a large maximum playlist length, the integer multiplication "playlist_max_length * MPD_PATH_MAX" may overflow. Change that to a division. This was not a dangerous bug, since it was only used for a quick estimate.
* ffmpeg: case AV_NOPTS_VALUE to int64_tMax Kellermann2008-12-241-2/+2
| | | | | The old code casted it to a 32 bit integer, which cut off bits. AVFormatContext.duration is a int64_t, so use this type.
* ffmpeg: don't assign "0" to pointerMax Kellermann2008-12-241-1/+1
| | | | Use NULL instead. Found by sparse.
* Makefile.am: fix sparse invocationMax Kellermann2008-12-241-1/+4
| | | | Added missing includes.
* added build test script "test.sh"Max Kellermann2008-12-241-0/+87
|
* mp3: "tag" argument is unused when libid3tag is disabledMax Kellermann2008-12-241-2/+2
| | | | Add G_GNUC_UNUSED attributes.
* listen: "port" argument is unused when TCP support is disabledMax Kellermann2008-12-241-1/+2
| | | | Add a G_GNUC_UNUSED attribute.
* Makefile.am: distribute only real scriptsMax Kellermann2008-12-241-1/+1
| | | | | Don't distribute the whole scripts/ directory, because this may include editor backup files and other temporary files.
* Makefile.am: removed m4/* from EXTRA_DISTMax Kellermann2008-12-241-1/+3
| | | | The m4 files are automatically included in the distribution.
* Makefile.am: don't add directory into EXTRA_DISTMax Kellermann2008-12-241-1/+1
| | | | Add separate files. This fixes "make distcheck".
* configure.ac: renamed shout optionsMax Kellermann2008-12-241-4/+4
| | | | configure options should use the dash instead of the underscore.
* configure.ac: fix --enable-un documentationMax Kellermann2008-12-241-2/+2
| | | | Unix domain socket support is enabled by default.
* configure.ac: disable the mikmod plugin by defaultMax Kellermann2008-12-242-1/+2
| | | | | | | | | | | | libmikmod seems to be unmaintained, and has several critical bugs which make MPD crash. http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=461519 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=476339 Disable the plugin by default to minimize harm. Users should explicitly enable the mikmod decoder (--enable-mod) if they wish to have it anyway.
* pcm_utils: check pcm_convert()==0Max Kellermann2008-12-243-5/+16
| | | | | | | | It is illegal to pass an empty audio buffer around. pcm_resample() sometimes seems to result in 0 samples, maybe related to libsamplerate. To work around that problem, add special checks after both pcm_convert() invocations. Removed the pcm_resample()==0 checks from pcm_convert().
* client: always attempt to flush deferred buffersMax Kellermann2008-12-241-2/+11
| | | | | | | | | | | When a response is very long (e.g. a large playlist > 100k songs), most of it will end up in the deferred buffers. Filling the deferred queue is very expensive currently, because a new buffer is allocated for every client_write() operation. This may lead to long delays, and the client might give up and disconnect meanwhile. This patch makes MPD attempt to flush the deferred queue as often as possible, to work around this problem. Due to the MPD 0.14 code freeze, we should not optimize the buffering code now.
* shout: fixed bad error message printoutViliam Mateicka2008-12-231-2/+2
|
* Build makefiles after checking available outputs and decoder pluginsQball Cow2008-12-221-10/+12
| | | | Move the AC_OUTPUT below the checks. So you nicely see what goes wrong.
* Merge branch 'http_buffer_remove' of git://git.musicpd.org/avuton/mpdMax Kellermann2008-12-213-12/+0
|\
| * Remove obsolete http_buffer* stuff that went away with the new curl backend.Avuton Olrich2008-12-203-12/+0
| |
* | Merge branch 'master' of git://git.musicpd.org/avuton/mpdMax Kellermann2008-12-216-224/+295
|\ \ | | | | | | | | | | | | | | | Conflicts: doc/mpdconf.example
| * | Fix AUTHORS UTF-8 compliance.Avuton Olrich2008-12-211-1/+1
| |/
| * Huge mpdconf update.Avuton Olrich2008-12-201-175/+198
| | | | | | | | | | | | | | Over time mpdconf has evolved and has been maintained and contributed to by many different folks at different times. Try to give it some unity. Attempt to clear up some language and make things crystal clear. Add more examples. Use tabs when it makes sense. Make better borders. Remove obsolete options. Try to put more defaults when it makes sense.
| * More clarification in the INSTALL fileAvuton Olrich2008-12-201-3/+4
| |
| * Word wrap to 80 char linesAvuton Olrich2008-12-202-31/+60
| |
| * Start off with a disclaimer about the NEWS file informationAvuton Olrich2008-12-201-0/+2
| |
| * Clarify some very messy documentation in the installation.Avuton Olrich2008-12-201-8/+23
| |
| * Update the text of the GPLv2, this fixes the address and a few other oddities.Avuton Olrich2008-12-201-14/+13
| |
* | doc: clarify "user" setting (starting as root / unprivileged)Max Kellermann2008-12-202-5/+8
| | | | | | | | | | The "user" option does not work if MPD is started by an unprivileged user.
* | log: map log level "SECURE" to GLib's "INFO"Max Kellermann2008-12-201-3/+3
|/ | | | | | Make "secure" a log level different from "default". "secure" should be right between "default" and "verbose". Map "default" to Glib's "MESSAGE" log level.
* MPD version 0.14~beta3release-0.14_beta3Avuton Olrich2008-12-201-1/+1
|
* configure.ac: check lame availability before shout_mp3 testMax Kellermann2008-12-171-5/+5
| | | | | If lame was not available, the shout_mp3 plugin was enabled anyway, and triggered compiler errors.
* configure.ac: disable shout when no encoder is foundMax Kellermann2008-12-171-12/+10
| | | | | First check if an ecoder plugin is available, then determine whether to enable the shout output plugin.
* Remove useless statement.Emanuele Giaquinta2008-12-171-1/+0
|
* player_control: check if errored_song is setMax Kellermann2008-12-171-4/+14
| | | | | | getPlayerErrorStr() assumes that pc.errored_song is set when an error occured. Since the song may have been deleted meanwhile, add a NULL check.
* playlist: clear pc.errored_song on deleteMax Kellermann2008-12-173-2/+23
| | | | | When a (remote) song is deleted from the playlist, there may still be a reference to it in pc.errored_song. Clear this reference.
* Remove useless computation. After the pthread_cond_wait loop there are at ↵Emanuele Giaquinta2008-12-171-2/+0
| | | | least MIN(od->bufferSize, size) free bytes in the buffer. Thus MIN(od->bufferSize - od->len, size) is always equal to MIN(od->bufferSize, size).
* Remove useless statement, curpos is initialized at the beginning of the loop.Emanuele Giaquinta2008-12-171-1/+0
|
* Factor computation.Emanuele Giaquinta2008-12-171-4/+6
|
* Use MIN.Emanuele Giaquinta2008-12-171-2/+2
|
* Call CloseComponent after AudioUnitUninitialize.Emanuele Giaquinta2008-12-171-1/+1
|
* man: document Unix domain socket configuration in the manualMax Kellermann2008-12-162-0/+7
| | | | Additionally, add an example in mpdconf.example.
* command: reject unsupported URI schemesMax Kellermann2008-12-161-1/+11
| | | | | | When a client-specified URI has a scheme which is not supported, do not try to open it as a local file, but provide a meaningful error message.
* ls: added uri_has_scheme()Max Kellermann2008-12-162-0/+10
| | | | | uri_has_scheme() checks if an URI contains the sub string "://", which makes MPD assume that it is a remote URI.
* ls: don't return suffix from parent directory nameMax Kellermann2008-12-161-1/+1
| | | | | When a file had no file name extension, getSuffix() could return the extension of the parent directory (if it had one).
* ls: reimplement getSuffix() with strrchr()Max Kellermann2008-12-161-8/+2
| | | | | The old getSuffix() code was quite wasteful, and can be replaced completely with strrchr().
* ls: use boolMax Kellermann2008-12-162-6/+7
| | | | Use the C99 "bool" data type instead of "int".
* ls: make printRemoteUrlHandlers() return voidMax Kellermann2008-12-163-5/+4
| | | | | printRemoteUrlHandlers() cannot fail, and does not need a return value.
* ls: removed isValidRemoteUtf8Url()Max Kellermann2008-12-165-46/+4
| | | | | The function didn't do anything useful, it was just several lines obfuscating that it was only forwarding isRemoteUrl()'s return value.
* ls: don't include time.hMax Kellermann2008-12-161-2/+0
| | | | ls.h does not need time.h, do not include it.
* ffmpeg: fixing ffmpeg_send_packet to allow multipacketsViliam Mateicka2008-12-151-16/+32
|