- 14 Mar, 2009 18 commits
-
-
Max Kellermann authored
On failure, the function should return NULL, not a boolean.
-
Max Kellermann authored
sparse cannot deal with C++.
-
Max Kellermann authored
sparse complains that a lot of limit.h macros are missing. Define them to keep sparse quiet.
-
Max Kellermann authored
Turn the "return false" error handlers into "return NULL".
-
Max Kellermann authored
Moved the hash table initialization from sticker_list_values() to the new function sticker_new(). This fixes a memory leak in sticker_list_values().
-
Max Kellermann authored
sticker_list_values() is only used internally in sticker.c. Remove sticker_song_list_values() completely, it is superseded by sticker_song_get().
-
Max Kellermann authored
-
Max Kellermann authored
The sticker struct can be used for enumerating values. This will replace the sticker_list_values() function.
-
Max Kellermann authored
Don't dump the volume of all mixers.
-
Max Kellermann authored
In some rare cases, there was a race condition between the output thread and the main thread: when you disable/enable an output device in the main thread, this caused a crash in the output thread. Protect the whole mixer struct with a GMutex to prevent that.
-
Max Kellermann authored
-
Max Kellermann authored
This patch allows the output plugins to import only mixer_list.h, instead of the full mixer_api.h (which would expose internal structures).
-
Max Kellermann authored
mixer_control.h should provide the functions needed to manipulate a mixer, without exposing the internal mixer API (which is provided by mixer_api.h).
-
Max Kellermann authored
-
Max Kellermann authored
-
Max Kellermann authored
-
Max Kellermann authored
Return true/false on success/failure, instead of 0/-1. Pass true/false instead of 1/0 for the "rel" boolean parameter.
-
Jochen Keil authored
Check for NULL not necessary here
-
- 13 Mar, 2009 1 commit
-
-
Avuton Olrich authored
This updates the copyright header to all be the same, which is pretty much an update of where to mail request for a copy of the GPL and the years of the MPD project. This also puts all committers under 'The Music Player Project' umbrella. These entries should go individually in the AUTHORS file, for consistancy.
-
- 12 Mar, 2009 10 commits
-
-
Jeffrey Middleton authored
-
Max Kellermann authored
When the destination chunk was empty in cross_fade_apply(), it had no audio_format attached (an attribute which is only used for assertion in the debug build). cross_fade_apply() should assign it the audio_format of the second chunk (if available), otherwise MPD will crash.
-
Max Kellermann authored
When there are chunks which are not yet finished, audio_output_all_check() returned the size of its music pipe minus one. I can't remember exactly why I subtracted 1 from the return value, it must have had something to do with a former meaning of this function. Now it induces assertion failures.
-
Max Kellermann authored
That linker flag got lost when AM_LDFLAGS was removed.
-
Max Kellermann authored
When an audio output device is disabled, also disable its mixer.
-
Max Kellermann authored
No caller must ever pass an invalid device number to mixer_control_setvol() or mixer_control_getvol().
-
Max Kellermann authored
-
Max Kellermann authored
-
Max Kellermann authored
Very small start..
-
Max Kellermann authored
-
- 11 Mar, 2009 6 commits
-
-
Eric Wollesen authored
-
Diego Elio 'Flameeyes' Pettenò authored
Passing libraries through LDFLAGS is a mistake that causes link to fail when using --as-needed. Since the ld arguments are positional, so are libtool's. Use the proper variable, thus, to pass the libraries.
-
Max Kellermann authored
-
Max Kellermann authored
Replaced both player_stop_decoder() invocations with player_dc_stop(), which also cleans up the pipe.
-
Max Kellermann authored
In the "CANCEL" command handler, the decoder is stopped twice: first by player_dc_stop(), then by dc_stop(). Remove the latter.
-
Max Kellermann authored
-
- 10 Mar, 2009 5 commits
-
-
Max Kellermann authored
This function was part of a workaround which we don't need anymore.
-
Jochen Keil authored
After adding the container_scan() method the update_regular_file() method was quite hard to read. Now there's update_container_file() which deals with container files. That way normal container files (i.e. without embedded tracks) are handled by the old code like a regular file. This will fix some of the odd behaviour observed.
-
Max Kellermann authored
snd_pcm_writei() returns the type snd_pcm_sframes_t, not int. Use the correct variable type.
-
Max Kellermann authored
If the PCM handle gets disconnected, don't close and clear it in alsa_recover(). The MPD core will call alsa_close() anyway. This way, we can always assume that alsa_data.pcm is always valid.
-
Max Kellermann authored
After a seek, wait until enough new chunks are decoded before starting playback. If this takes too long, send silence chunks to the audio outputs meanwhile.
-