aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* decoder_thread: fix CUE track playbackMax Kellermann2010-01-162-5/+11
| | | | | | | | | | | | The patch "input/file: don't fall back to parent directory" introduced a regression: when trying to play a CUE track, decoder_run_song() tries to open the file as a stream first, but this fails, because the path is virtual. This patch fixes decoder_run_song() (instead of reverting the previous patch) to accept input_stream_open() failures if the song is a local file. It passes the responsibility to handle non-existing files to the decoder's file_decode() method.
* NEWS: added missing entryMax Kellermann2010-01-161-0/+1
|
* dbUtils: return empty tag value only if no value was foundMax Kellermann2010-01-021-1/+4
| | | | | | | | This fixes a regression in the patch "return multiple tag values per song": even when the song has values for the specified tag type, the empty string gets added to the set, because the "return" was removed. This patch adds a flag which remembers whether at least one value was found.
* input/curl: removed the built-in rewinding codeMax Kellermann2009-12-301-181/+10
| | | | This has been reimplemented in the "rewind" input plugin.
* input/rewind: new input_stream wrapper to allow stream rewindingMax Kellermann2009-12-297-1/+318
| | | | | | | | | | | | This replaces the rewinding buffer code from the CURL input plugin. It is more generic, and allows rewinding even when the server sends Icy-Metadata (which would have been too difficult to implement within the CURL plugin). This is a rather complex patch for the stable branch (v0.15.x), but it fixes a serious problem: the "vorbis" decoder plugin was unable to play streams with Icy-Metadata, because it couldn't rewind the stream after detecting the codec (Vorbis vs. FLAC).
* decoder/{ffmpeg,flac,vorbis}: added more flac/vorbis MIME typesMax Kellermann2009-12-295-4/+24
| | | | | Support deprecated MIME types such as "audio/x-ogg". Support new types such as "audio/flac".
* Modify version string to post-release version 0.15.8~gitAvuton Olrich2009-12-272-1/+4
|
* mpd version 0.15.7release-0.15.7Avuton Olrich2009-12-272-2/+2
|
* dbUtils: return multiple tag values per songMax Kellermann2009-12-272-1/+1
| | | | | | When collecting tag values for the result set, add all of a song's tag values of the searched type. This affects the "list" command. Previously, "list" only considered the first tag value of a song.
* decoder_thread: eliminate jitter after seek failureMax Kellermann2009-12-262-1/+6
| | | | | | Don't clear the music pipe when seeking has failed - check the "seeking" flag instead of "command==SEEK". Clear the "seeking" flag in decoder_seek_error().
* decoder/wavpack: allow fine-grained seekingMax Kellermann2009-12-262-3/+3
| | | | | First multiply the floating point return value of decoder_seek_where(), then cast to integer.
* Makefile.am: create doxygen output directoryMax Kellermann2009-12-261-0/+1
|
* iso, zip: fixed memory leak in destructorMax Kellermann2009-12-153-2/+5
| | | | Free the "context" pointer in the method archive_plugin.close().
* archive: close archive when stream is closedMax Kellermann2009-12-155-0/+11
| | | | | | | | Fixes a memory leak: the "archive" input plugin opens the archive, but never closes it. This patch moves the responsibility for doing that to archive_plugin.open_stream(). This is an slight internal API change, but it is the simplest and least intrusive fix for the memory leak.
* archive/bz2: removed NULL check before g_free()Max Kellermann2009-12-151-2/+2
| | | | g_free(NULL) is allowed.
* input/archive: close the archive file on errorMax Kellermann2009-12-152-0/+2
| | | | Fixed memory leak in error handler.
* test/run_input: deinitialize everything after open() errorMax Kellermann2009-12-151-40/+50
| | | | This enables valgrind debugging after an error occurred.
* test/run_input: initialize archive pluginsMax Kellermann2009-12-151-0/+15
| | | | Enable archive plugin debugging.
* input/file: don't fall back to parent directoryMax Kellermann2009-12-152-18/+5
| | | | | This code has never made any sense, and has broken some of the archive plugin.
* decoder/wavpack: don't use the nonstandard "uchar" typeMax Kellermann2009-12-111-1/+1
| | | | Use the signed C99 type int8_t instead.
* mixer: explicitly close all mixers on shutdownMax Kellermann2009-12-082-0/+5
| | | | | | Mixers with the "global" flag set aren't closed automatically when the output device is closed. Thus, they might still be open when MPD shuts down.
* mapper: apply filesystem_charset to playlistsMax Kellermann2009-12-085-11/+37
| | | | | | | | This fixes an inconsistency in the stored playlist subsystem: when obtaining the list of playlists (listplaylist, listplaylistinfo), the file names in the playlist directory are converted to UTF-8 (according to filesystem_charset), but when saving or loading playlists, the filesystem_charset setting was ignored.
* command: verify playlist name in the "rm" commandMax Kellermann2009-12-082-0/+4
| | | | Call spl_valid_name() in spl_delete().
* mapper: fix memory leak when playlist_directory is not setMax Kellermann2009-12-082-2/+3
| | | | Don't allocate the file name before the playlist_dir==NULL check.
* tag_id3: fix ID3v1 charset conversionsvitoos2009-11-302-1/+3
| | | | | If we define id3v1_encoding, then the tags are not added to the database.
* ffmpeg: don't try to force stereoMax Kellermann2009-11-302-4/+2
| | | | | | The plugin code tried to force libavcodec to supply stereo samples. That however has never actually worked. By removing this code, we are able to play surround files for the first time.
* decoder/flac: fixed compiler warningMax Kellermann2009-11-191-3/+1
| | | | | | Removed the "vtrack" local variable (which triggered a gcc warning because it was after the newly introduced NULL check), and run strtol() on the original parameter.
* Modify version string to post-release version 0.15.7~gitAvuton Olrich2009-11-182-1/+4
|
* mpd version 0.15.6release-0.15.6Avuton Olrich2009-11-182-2/+2
|
* decoder/flac: fixed NULL pointer dereference in CUE codeMax Kellermann2009-11-182-0/+3
| | | | The function flac_vtrack_tnum() was missing a strrchr()==NULL check.
* id3: allow 4 MB RIFF/AIFF tagsMax Kellermann2009-11-152-1/+3
| | | | | | | Allow RIFF/AIFF ID3 tags up to 4 MB (old limit was 256 kB). This might still be too small for some users, and when somebody complains, we might do something more clever (like streaming the data into libid3tag?).
* decoder/ffmpeg: align the output bufferMax Kellermann2009-11-152-5/+25
| | | | | | On some platforms, libavcodec wants the output buffer aligned to 16 bytes (because it uses SSE/Altivec internally). It will segfault when you don't obey this rule.
* decoder/flac: fixed CUE seeking range checkMax Kellermann2009-11-112-14/+9
| | | | | | If flac_container_decode() gets a seek destination which is out of range, it ignores the SEEK command (never finishes it). This leads to MPD lockup, because the player thread waits for completion.
* oggflac: rewind stream after FLAC detectionMax Kellermann2009-11-112-0/+9
| | | | | The oggflac plugin has been completely broken for quite a while and nobody has noticed - maybe we should remove it?
* zzip: require libzzip 0.13Max Kellermann2009-11-102-1/+3
| | | | We need the function zzip_file_stat().
* input/mms: require libmms 0.4Max Kellermann2009-11-102-1/+2
| | | | We're using API functions which are not available in 0.3.
* sticker: added fallback for sqlite3_prepare_v2()Max Kellermann2009-11-102-0/+5
| | | | This function was not present in SQLite < 3.4.
* input/lastfm: fixed variable name in GLib<2.16 code pathMax Kellermann2009-11-102-1/+3
| | | | Should be "lastfm_user", not "lastfm_username".
* song_save: increased maximum line length to 32 kBMax Kellermann2009-11-012-3/+8
| | | | | | | The line buffer had a fixed size of 5 kB, and was allocated on the stack. This was too small for some users. As a hotfix, we're increasing the buffer size to 32 kB now, allocated on the heap. In MPD 0.16, we'll switch to dynamic allocation.
* decoder_control: removed unused DECODE_TYPE macrosMax Kellermann2009-10-311-3/+0
|
* decoder/ffmpeg: convert metadataMax Kellermann2009-10-282-4/+6
| | | | | | Convert the metadata with the libavformat function av_metadata_conv(). This ensures that canonical tag names are provided by libavformat, and we can remove the "artist" vs "author" workaround.
* update: delete ignored symlinks from databaseMax Kellermann2009-10-272-1/+6
| | | | | | When you disable the "follow_outside_symlinks" or the "follow_inside_symlinks" setting, the next update should remove the now-ignored files from the database.
* output_thread: check again if output is open on PAUSEMax Kellermann2009-10-212-0/+10
| | | | | Basically the same as the 0.15.5 patch "check again if output is open on CANCEL". Same race condition, same fix.
* Modify version string to post-release version 0.15.6~gitAvuton Olrich2009-10-182-1/+4
|
* mpd version 0.15.5release-0.15.5Avuton Olrich2009-10-182-2/+2
|
* decoder/flac: fixed two memory leaks in the CUE tag loaderMax Kellermann2009-10-162-2/+5
| | | | | | Don't initialize "vc" and "cs" with FLAC__metadata_object_new(); that value is overwritten by FLAC__metadata_get_tags() and FLAC__metadata_get_cuesheet().
* update: fixed memory leak during container scanMax Kellermann2009-10-162-3/+7
| | | | The return value of map_directory_child_fs() must be freed.
* update: song_file_new() cannot failMax Kellermann2009-10-161-3/+0
| | | | | Removed the NULL check. If that NULL check was correct, that would have been a memory leak (vtrack).
* output_thread: check again if output is open on CANCELMax Kellermann2009-10-162-1/+3
| | | | | | | | When the player thread unpauses, it sends CANCEL to the output thread, after having checked that the output is still open. Problem is when the output thread closes the device before it can process the CANCEL command - race condition. This patch adds another "open" check inside the output thread.
* input/curl: fixed endless loop during bufferingMax Kellermann2009-10-132-1/+2
| | | | | | | When the connection is lost while buffering, the CURL input plugin may enter an endless loop, because it does not check the EOF condition. This patch makes fill_buffer() return success only if there's at least one buffer, which is enough of a check.x