aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* decoder/mp4ff: moved code to mp4ff_tag_name_parse()Max Kellermann2010-06-301-1/+7
|
* tag_ape: remove duplicate entries in the tag name tableMax Kellermann2010-06-301-7/+5
| | | | Reuse the function tag_name_parse_i().
* tag: added function tag_name_parse()Max Kellermann2010-06-303-12/+60
| | | | Convert a string into a tag_type enum.
* tag_ape: move code to tag_ape_name_parse()Max Kellermann2010-06-301-1/+7
|
* decoder/vorbis: use single global ov_callbacks constantMax Kellermann2010-06-301-7/+9
| | | | Initialize the ov_callbacks struct at compile time.
* decoder/mp4ff: support tags "albumartist", "band"Max Kellermann2010-06-301-0/+2
| | | | | I'm not sure if mapping "band" to TAG_PERFORMER is correct, but it might be better than nothing.
* decoder/mp4ff: use tag_table.h to parse tag namesMax Kellermann2010-06-301-18/+15
| | | | Convert if/else/else/... to a loop.
* tag_ape: move table lookup to tag_table.hMax Kellermann2010-06-302-9/+52
| | | | Allow code sharing.
* tag_ape: support album artistMax Kellermann2010-06-301-0/+1
| | | | | | | | | | I took this tag name from a MusePack sample file I got from a user. It is not documented in the APE specification: http://wiki.hydrogenaudio.org/index.php?title=APE_key People seem to be using undocumented extensions to the specification anyway, and the best we can do is attempt to support them.
* tag_ape: simplified the apeItems arrayMax Kellermann2010-06-301-22/+12
| | | | | Make "enum tag_type" the array index, and convert apeItems to a sparse array.
* tag_ape: moved code to tag_ape_import_item()Max Kellermann2010-06-301-12/+22
| | | | Improve code readability.
* tag_ape: converted apeItems and tagItems to global varsMax Kellermann2010-06-301-20/+20
| | | | Don't initialize those arrays each time tag_ape_load() is called.
* decoder/mad: fix buffer variable name on !HAVE_ID3TAGMax Kellermann2010-05-301-3/+3
|
* input/mms: initialize the "eof" attributeMax Kellermann2010-05-181-0/+2
|
* input/mms: fix memory leak in error handlerMax Kellermann2010-05-181-0/+1
|
* decoder/mad: properly calculate ID3 size without libid3tagMax Kellermann2010-04-131-5/+25
| | | | | | | | | | Without libid3tag, we were trying to skip the ID3 frame (since 0.15.2). Its length however was not calculated at all, we were just dropping everything from the current input buffer. This lead to the first few seconds of the file being skipped. This patch attempts to calculate the ID3v2 frame size with the formula from: http://www.id3.org/id3v2.4.0-structure 3.1 and 6.2
* decoder/mpcdec: fix replay gain formula with v8Aleksei Kaveshnikov2010-03-191-0/+8
| | | | | | | | | | "When playing musepack files with mpd v0.15.8, rg seems to have no effect. Using sample file below, mpd says 'computing ReplayGain album scale with gain 122.879997, peak 0.549150'. One thing though, if I build mpd against old libmpcdec-1.2.6, rg works as expected: 'computing ReplayGain album scale with gain 16.820000, peak 0.099765'"
* player_thread: postpone song tags during cross-fadeMax Kellermann2010-03-171-0/+27
| | | | | | | | | Previously, tags of the new song being cross-faded in were sent immediately. That can cause wrong information being displayed, because the "previous" song might send its tag at the end again, overriding the "next" song's tag. This patch saves & merges the tag of the next song, and sends it when cross-fading is finished, and the next song really starts.
* tag: added function tag_merge_replace()Max Kellermann2010-03-173-21/+26
| | | | | Like tag_merge(), but can deal with NULL parameters, and frees both tag objects.
* decoder/mpcdec: fix negative shift on fixed-point samplesPiotr Gozdur2010-03-171-1/+1
| | | | | | "There is a bug in fixed-point musepack (musepack_src_r435) playback. In floating-point audio is OK but in fixed audio is distorted. I have made a patch for this"
* playlist: fix single+repeat in random modeMax Kellermann2010-03-071-1/+2
| | | | | | | With single+repeat enabled, it is expected that MPD repeats the current song over andd over. With random mode also enabled, this didn't work, because the song order was shuffled internally. This patch adds a special check for this case.
* decoder/mad: fix crash when seeking at end of songMax Kellermann2010-02-271-4/+0
| | | | | | Removed the decoder_command_finished() call at the end of mp3_decode(). This is invalid, because decoder_command_finished() has already been called in mp3_read().
* decoder/ffmpeg: added more MIME typesMax Kellermann2010-01-171-0/+6
| | | | Taken from the ffmpeg sources.
* decoder/ffmpeg: append file name suffix to virtual stream URLMax Kellermann2010-01-171-4/+28
| | | | | | | To allow libavformat to detect the format of the input file, append the suffix of the input file to the URL of the virtual stream. This specifically enables the "shorten" codec, which is supported by libavformat/raw.c, detected only by the suffix.
* queue: don't repeat current song in consume modeMax Kellermann2010-01-161-2/+2
| | | | | Check consume mode in queue_next_order(), because the current song would be deleted as soon as it's finished; it cannot be played again.
* decoder_thread: fix CUE track playbackMax Kellermann2010-01-161-5/+10
| | | | | | | | | | | | 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.
* 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-295-0/+312
| | | | | | | | | | | | 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-294-4/+22
| | | | | Support deprecated MIME types such as "audio/x-ogg". Support new types such as "audio/flac".
* dbUtils: return multiple tag values per songMax Kellermann2009-12-271-1/+0
| | | | | | 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-261-1/+5
| | | | | | 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-261-3/+2
| | | | | First multiply the floating point return value of decoder_seek_where(), then cast to integer.
* iso, zip: fixed memory leak in destructorMax Kellermann2009-12-152-2/+4
| | | | Free the "context" pointer in the method archive_plugin.close().
* archive: close archive when stream is closedMax Kellermann2009-12-154-0/+9
| | | | | | | | 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-151-0/+1
| | | | Fixed memory leak in error handler.
* input/file: don't fall back to parent directoryMax Kellermann2009-12-151-18/+3
| | | | | 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-081-0/+4
| | | | | | 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-084-11/+36
| | | | | | | | 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-081-0/+3
| | | | Call spl_valid_name() in spl_delete().
* mapper: fix memory leak when playlist_directory is not setMax Kellermann2009-12-081-2/+2
| | | | Don't allocate the file name before the playlist_dir==NULL check.
* tag_id3: fix ID3v1 charset conversionsvitoos2009-11-301-1/+1
| | | | | If we define id3v1_encoding, then the tags are not added to the database.
* ffmpeg: don't try to force stereoMax Kellermann2009-11-301-4/+0
| | | | | | 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.
* decoder/flac: fixed NULL pointer dereference in CUE codeMax Kellermann2009-11-181-0/+2
| | | | The function flac_vtrack_tnum() was missing a strrchr()==NULL check.
* id3: allow 4 MB RIFF/AIFF tagsMax Kellermann2009-11-151-1/+1
| | | | | | | 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-151-5/+24
| | | | | | 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-111-14/+8
| | | | | | 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.