aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/pcm/SoxrResampler.cxx31
1 files changed, 18 insertions, 13 deletions
diff --git a/src/pcm/SoxrResampler.cxx b/src/pcm/SoxrResampler.cxx
index 4b914cea2..be2893ffa 100644
--- a/src/pcm/SoxrResampler.cxx
+++ b/src/pcm/SoxrResampler.cxx
@@ -34,7 +34,12 @@ static constexpr Domain soxr_domain("soxr");
static constexpr unsigned long SOXR_DEFAULT_RECIPE = SOXR_HQ;
-static unsigned long soxr_quality_recipe = SOXR_DEFAULT_RECIPE;
+/**
+ * Special value for "invalid argument".
+ */
+static constexpr unsigned long SOXR_INVALID_RECIPE = -1;
+
+static unsigned long soxr_quality_recipe;
static const char *
soxr_quality_name(unsigned long recipe)
@@ -55,39 +60,39 @@ soxr_quality_name(unsigned long recipe)
gcc_unreachable();
}
-static bool
+gcc_pure
+static unsigned long
soxr_parse_converter(const char *converter)
{
assert(converter != nullptr);
assert(memcmp(converter, "soxr", 4) == 0);
if (converter[4] == '\0')
- return true;
+ return SOXR_DEFAULT_RECIPE;
if (converter[4] != ' ')
- return false;
+ return SOXR_INVALID_RECIPE;
// converter example is "soxr very high", we want the "very high" part
const char *quality = converter + 5;
if (strcmp(quality, "very high") == 0)
- soxr_quality_recipe = SOXR_VHQ;
+ return SOXR_VHQ;
else if (strcmp(quality, "high") == 0)
- soxr_quality_recipe = SOXR_HQ;
+ return SOXR_HQ;
else if (strcmp(quality, "medium") == 0)
- soxr_quality_recipe = SOXR_MQ;
+ return SOXR_MQ;
else if (strcmp(quality, "low") == 0)
- soxr_quality_recipe = SOXR_LQ;
+ return SOXR_LQ;
else if (strcmp(quality, "quick") == 0)
- soxr_quality_recipe = SOXR_QQ;
+ return SOXR_QQ;
else
- return false;
-
- return true;
+ return SOXR_INVALID_RECIPE;
}
bool
pcm_resample_soxr_global_init(const char *converter, Error &error)
{
- if (!soxr_parse_converter(converter)) {
+ soxr_quality_recipe = soxr_parse_converter(converter);
+ if (soxr_quality_recipe == SOXR_INVALID_RECIPE) {
error.Format(soxr_domain,
"unknown samplerate converter '%s'", converter);
return false;