diff options
author | Max Kellermann <max@duempel.org> | 2014-10-25 21:25:49 +0200 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2014-10-25 22:06:08 +0200 |
commit | d7f024c51047222dda4224c56865d66c8f384c76 (patch) | |
tree | 370b3be47b695aa16eef7f7bd96737cc83cbfe17 /src | |
parent | bc5a53574c64eb51d2cbf31b26f4607e566eeaab (diff) | |
download | mpd-d7f024c51047222dda4224c56865d66c8f384c76.tar.gz mpd-d7f024c51047222dda4224c56865d66c8f384c76.tar.xz mpd-d7f024c51047222dda4224c56865d66c8f384c76.zip |
OutputThread: fall back to PCM if given DSD sample rate is not supported
Works around the "PCM conversion from f to dsd is not implemented"
error message that prevents DSD playback.
Diffstat (limited to 'src')
-rw-r--r-- | src/output/OutputThread.cxx | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/src/output/OutputThread.cxx b/src/output/OutputThread.cxx index 54664bb65..2ec0670c1 100644 --- a/src/output/OutputThread.cxx +++ b/src/output/OutputThread.cxx @@ -22,6 +22,7 @@ #include "OutputAPI.hxx" #include "Domain.hxx" #include "pcm/PcmMix.hxx" +#include "pcm/Domain.hxx" #include "notify.hxx" #include "filter/FilterInternal.hxx" #include "filter/plugins/ConvertFilterPlugin.hxx" @@ -165,6 +166,10 @@ AudioOutput::Open() out_audio_format.ApplyMask(config_audio_format); mutex.unlock(); + + const AudioFormat retry_audio_format = out_audio_format; + + retry_without_dsd: success = ao_plugin_open(this, out_audio_format, error); mutex.lock(); @@ -189,6 +194,31 @@ AudioOutput::Open() mutex.unlock(); ao_plugin_close(this); + + if (error.IsDomain(pcm_domain) && + out_audio_format.format == SampleFormat::DSD) { + /* if the audio output supports DSD, but not + the given sample rate, it asks MPD to + resample; resampling DSD however is not + implemented; our last resort is to give up + DSD and fall back to PCM */ + + // TODO: clean up this workaround + + FormatError(output_domain, "Retrying without DSD"); + + out_audio_format = retry_audio_format; + out_audio_format.format = SampleFormat::FLOAT; + + /* clear the Error to allow reusing it */ + error.Clear(); + + /* sorry for the "goto" - this is a workaround + for the stable branch that should be as + unintrusive as possible */ + goto retry_without_dsd; + } + CloseFilter(); mutex.lock(); |