From 8b366f079524f678ff39f862d8a808f98ef0afe2 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 21 Jan 2015 19:55:48 +0100 Subject: pcm/SoxrResampler: soxr_parse_converter() returns unsigned long --- src/pcm/SoxrResampler.cxx | 31 ++++++++++++++++++------------- 1 file changed, 18 insertions(+), 13 deletions(-) (limited to 'src/pcm') 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; -- cgit v1.2.3