aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* player: removed "volatile" attributesMax Kellermann2008-11-031-6/+6
| | | | | Removed the "volatile" attributes from several variables which are not important for synchronization.
* player: no CamelCaseMax Kellermann2008-11-033-45/+45
| | | | | Renamed variables and internal functions. Most of the player_control.h API remains in CamelCase for now.
* decoder: converted DECODE_ERROR_* to enumMax Kellermann2008-11-031-4/+6
|
* decoder: removed "volatile" modifierMax Kellermann2008-11-031-2/+2
| | | | | | The variable "next_song" is already protected by a memory barrier. "total_time" is not important for synchronization, and we don't need "volatile" here.
* decoder: no CamelCaseMax Kellermann2008-11-035-31/+31
| | | | Renamed variables and functions.
* decoder_api: send song tag in decoder_data()Max Kellermann2008-11-031-8/+26
| | | | | Before passing the first chunk to the audio output device, send the current song's tag.
* decoder_api: automatically send stream tagMax Kellermann2008-11-035-51/+60
| | | | | | | | | | If an input stream provides tags (e.g. from an icecast server), send them in the decoder_data() and decoder_tag() methods. Removed the according code from the mp3 and oggvorbis plugins - decoders shouldn't have to care about stream tags. This patch also adds the missing decoder_tag() invocation to the mp3 plugin.
* tag: added function tag_has_type()Max Kellermann2008-11-032-0/+19
| | | | | tag_has_type() checks whether the tag contains one or more items with the specified type.
* decoder_api: no CamelCaseMax Kellermann2008-11-031-34/+32
| | | | Renamed variables.
* configure.ac: disable libflac's oggflac support with --disable-oggflacAvuton Olrich2008-11-031-1/+1
| | | | The configure flag "--disable-oggflac" disabled only liboggflac.
* configure.ac: AC_SUBST(MP4FF_LIBS)Max Kellermann2008-11-031-0/+1
| | | | The AC_SUBST() call was missing in the previous patch.
* configure.ac: separate the "aac" and "mp4" decoder testsMax Kellermann2008-11-033-5/+48
| | | | | | | MPD used to have a copy of the mp4ff library. Since that has been removed, AAC suport was disabled when there was no libmp4ff. Separate the libmp4ff test, and enable AAC support no matter if libmp4ff is available.
* alsa: initialize "device" with NULLMax Kellermann2008-11-031-2/+1
| | | | | When using autodetection, AlsaData.device wasn't properly initialized with NULL. This broke autodetection randomly.
* mod: removed boolean globals for lazy initMax Kellermann2008-11-031-15/+4
| | | | | The variables mod_mikModInitiated and mod_mikModInitError were used to control lazy initialization, but they are superfluous now.
* mod: always initialize libmikmodMax Kellermann2008-11-031-8/+1
| | | | | | | The "mod" decoder plugin was being initialized lazily, but was deinitialized unconditionally. That led to segmentation faults. Convert mod_initMikMod() to be the global module initialization method. The MPD core should care about lazy initialization.
* output: don't allow length==0Max Kellermann2008-11-023-0/+5
| | | | | Nobody should call playAudio() with an empty chunk. Add some assertions on that.
* player: don't play empty chunksMax Kellermann2008-11-021-0/+3
| | | | | An empty chunk may happen when it only contains a tag, but no PCM data. Don't call playAudio() then.
* ffmpeg: fix boolean inversion in ffmpeg_tag()Max Kellermann2008-11-021-1/+1
| | | | | | ffmpeg_tag() deleted the tag when ffmpeg_helper() returned success. The return value was interpreted incorrectly, it should return the tag on success.
* removed unused sourcesMax Kellermann2008-11-026-691/+0
| | | | | Cleaning up artifacts from a merge gone wrong. Also remove the ringbuf library, which is not being used.
* player: copy stream tag to the song structMax Kellermann2008-11-021-5/+19
| | | | | | Non-local songs used to have no tags. If the decoder sends us a tag, we should incorporate it into the song struct. This way, clients can always show the correct song name (if provided by the server).
* player: added player.songMax Kellermann2008-11-021-0/+7
| | | | | Always remember which song is currently being sent to the audio device.
* ffmpeg: don't rewind stream in url_close()Max Kellermann2008-11-021-4/+0
| | | | | Rewinding the stream here is not useful, but may consume valuable resources (and time).
* decoder: rewind input stream after try_decode()Max Kellermann2008-11-023-9/+8
| | | | | | | | | The try_decode() method may have read some data from the stream, which is now lost. To make this data available to other methods, get it back by rewinding the input stream after each try_decode() invocation. The ogg and wavpack plugins did this manually and inconsistently; this code can now be removed.
* decoder: moved code to decoder_try_decode()Max Kellermann2008-11-021-6/+13
|
* player: send chunk tag to audio deviceMax Kellermann2008-11-021-0/+3
| | | | | If a chunk contains a tag, send it to the audio output device. Few output plugins support this, e.g. shout has support for sending tags.
* crossfade: copy tagMax Kellermann2008-11-021-0/+5
| | | | | | If the source chunk has a tag, merge it into the destination chunk. The source chunk gets deleted after that, and this is our last chance to grab the tag.
* ffmpeg: use decoder_read() wrapper instead of direct input_stream_read()Max Kellermann2008-11-021-1/+2
| | | | | decoder_read() checks the decoder command. Without this patch, the ffmpeg plugin could become unresponsive.
* ogg, ffmpeg: try to decode, even when the stream is not seekableMax Kellermann2008-11-023-11/+1
| | | | | | | Ogg and ffmpeg detection was disabled when the stream was not seekable, because the detection was too expensive. Since the curl input stream can now rewind the stream cheaply, we can re-enable detection on streams.
* input_curl: buffered rewindingMax Kellermann2008-11-021-3/+115
| | | | | | | | During codec detection, the beginning of the stream is consumed. This is a common operation, which takes a lot of time when handling remote resources. To optimize this, remember the first 64 kB of a stream. This way, we can rewind the stream without actually fetching the start of the stream again.
* decoder_api: added decoder_tag()Max Kellermann2008-11-023-2/+30
| | | | Provide an API for submitting additional tags from the stream.
* aac, mod: moved decoder command check into loop conditionMax Kellermann2008-11-022-18/+9
| | | | | Avoid while(true) loops, and convert them to a loop with a proper condition.
* aac, mod: don't check for SEEK commandMax Kellermann2008-11-022-18/+2
| | | | | | Since the aac and mod plugins have told MPD that they cannot seek, MPD will never send a SEEK command to them. Removed the SEEK comand checks from both plugins.
* decoder_api: pass "seekable" flag to decoder_initialized()Max Kellermann2008-11-0215-44/+37
| | | | | | Don't pass the "seekable" flag with every decoder_data() invocation. Since that flag won't change within the file, it is enough to pass it to decoder_initialized() once per file.
* music_pipe: add tag pointer to the music_chunk structMax Kellermann2008-11-022-1/+41
| | | | | | Each music chunk can now carry a tag object. Decoder plugins which support it (e.g. oggvorbis) may use this to inject decoded tags into their output.
* music_pipe: added functions chunk_init() and chunk_free()Max Kellermann2008-11-021-10/+38
| | | | | These two functions will care about memory allocation and deallocation in the future.
* music_pipe: document struct music_chunkMax Kellermann2008-11-021-0/+11
| | | | Add doxygen compatible comments.
* music_pipe: removed "volatile"Max Kellermann2008-11-021-2/+2
| | | | | The "volatile" keyword doesn't help here, because we have proper memory barriers, but it disables some optimizations. Remove it.
* music_pipe: renamed "size" to "num_chunks"Max Kellermann2008-11-022-13/+12
| | | | The name "num_chunks" expresses the meaning of the variable better.
* music_pipe: no CamelCaseMax Kellermann2008-11-024-29/+29
| | | | Rename all variables and struct members.
* music_pipe: set bit_rate and time in music_pipe_append()Max Kellermann2008-11-021-10/+10
| | | | Don't bother to pass these values as parameters to tail_chunk().
* music_pipe: moved code to music_chunk_append()Max Kellermann2008-11-021-13/+28
|
* music_pipe: renamed "ob" to "music_pipe"Max Kellermann2008-11-022-49/+49
| | | | Last music_pipe rename patch: renamed the global variable (singleton).
* music_pipe: more wrapper functionsMax Kellermann2008-11-022-8/+33
| | | | | | Replace all direct music_pipe struct accesses with wrapper functions. The compiled machine code is the same, but this way, we can change struct internals more easily.
* music_pipe: use GLib instead of utils.hMax Kellermann2008-11-021-3/+3
| | | | Eliminate the deprecated utils.h memory allocation functions.
* decoder: replaced music_pipe.audioFormat with dc.out_audio_formatMax Kellermann2008-11-026-22/+34
| | | | | | | .. and rename dc.audioFormat to dc.in_audio_format. The music pipe does not need to know the audio format, and its former "audioFormat" property indicated the format of the most recently added chunk, which might be confusing when you are reading the oldest chunks.
* music_pipe: pass frame size to tail_chunk()Max Kellermann2008-11-021-3/+2
| | | | Don't make tail_chunk() calculate the frame size again.
* music_pipe: renamed ob_* functions to music_pipe_*Max Kellermann2008-11-026-55/+56
| | | | Rename all functions to the new prefix.
* music_pipe: renamed struct output_buffer to struct music_pipeMax Kellermann2008-11-025-19/+24
| | | | .. and rename ob_chunk to struct music_chunk.
* renamed outputBuffer.[ch] to pipe.[ch]Max Kellermann2008-11-028-10/+10
| | | | | | | No CamelCase in the file name. The output_buffer struct is going to be renamed to music_pipe. There are so many buffer levels in MPD, and calling this one "output buffer" is wrong, because it's not the last buffer before the music reaches the output devices.
* decoder: enable decoders even if they have no init() methodMax Kellermann2008-11-021-1/+1
| | | | | | Commit 1a4a3e1f moved decoders into a static array, but failed to enable those plugins who did not have an init() method at all. This patch corrects the "enabled" check.