| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
The MPD core logs the audio format of all audio outputs. Remove the
duplicate message from the plugins.
|
| |
|
|
|
|
|
| |
Use GLib's GError library for reporting output device failures.
Note that some init() methods don't clean up properly after a failure,
but that's ok for now, because the MPD core will abort anyway.
|
| |
|
|
|
|
|
| |
The MPD core guarantees that the audio_output object is always
consistent, and our pa_simple!=NULL checks are superfluous. Also
don't manually close the device on error in pulse_play(), since the
MPD core does this automatically when the play() method returns 0.
|
| |
|
|
|
|
| |
audio_output_get_name() has been removed, which was the only function
left in output_api.h. The output plugin doesn't need the audio_output
object at all, remove the parameter from the init() method.
|
| |
|
|
| |
Use config_get_block_string("name") instead of audio_output_get_name().
|
| |
|
|
|
|
| |
The meaning of the chunk depends on the audio format; don't suggest a
specific format by declaring the pointer as "char*", pass "void*"
instead.
|
| |
|
|
|
|
|
|
| |
The old API required an output plugin to not return until all data
passed to the play() method is consumed. Some output plugins have to
loop to fulfill that requirement, and may block during that. Simplify
these, by letting them consume only part of the buffer: make play()
return the length of the consumed data.
|
| |
|
|
|
| |
All config_get_block_*() functions should accept constant config_param
pointers.
|
| |
|
|
| |
This replaces lots of getBlockParam() invocations.
|
| |
|
|
| |
Renamed functions, types, variables.
|
| |
|
|
| |
Use GLib's G_GNUC_UNUSED instead of gcc.h's mpd_unused.
|
| |
|
|
|
| |
The output thread automatically waits some time before retrying to
open the device. Don't duplicate this check in the pulse plugin.
|
| |
|
|
|
| |
The pulse plugin crashed with a segmentation fault when the pulse
server was killed.
|
| |
|
|
|
| |
Commit d692e9 broke configurations without explicit server or sink
configuration. Check for getBlockParam()==NULL.
|
| |
|
|
|
| |
Don't return 0/-1 on success/error, but true/false. Instead of int,
use bool for storing flags.
|
| |
|
|
|
| |
The pointer becomes invalid due to pa_simple_free(), which may lead to
segmentation faults when the output is reopened later.
|
| |
|
|
| |
Use GLib allocation and logging functions.
|
| | |
|
| |
|
|
| |
Don't compile the sources of disabled output plugins at all.
|
|
|
Again, no CamelCase in the directory name.
|
