From 3bc4224bfda5ebf9cd6969ea0f6647b1af3e9da6 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 11 Feb 2009 18:00:48 +0100 Subject: audio_format: added validation functions In addition to audio_format_valid(), provide functions which validate only one attribute of an audio_format. These functions are reused by audio_format_parse(). --- src/audio_format.h | 37 ++++++++++++++++++++++++++++++++++--- 1 file changed, 34 insertions(+), 3 deletions(-) (limited to 'src/audio_format.h') diff --git a/src/audio_format.h b/src/audio_format.h index 739fe6c1..a7ff3083 100644 --- a/src/audio_format.h +++ b/src/audio_format.h @@ -40,15 +40,46 @@ static inline bool audio_format_defined(const struct audio_format *af) return af->sample_rate != 0; } +/** + * Checks whether the sample rate is valid. + * + * @param sample_rate the sample rate in Hz + */ +static inline bool +audio_valid_sample_rate(unsigned sample_rate) +{ + return sample_rate > 0 && sample_rate < (1 << 30); +} + +/** + * Checks whether the sample format is valid. + * + * @param bits the number of significant bits per sample + */ +static inline bool +audio_valid_sample_format(unsigned bits) +{ + return bits == 16 || bits == 24 || bits == 8; +} + +/** + * Checks whether the number of channels is valid. + */ +static inline bool +audio_valid_channel_count(unsigned channels) +{ + return channels == 1 || channels == 2; +} + /** * Returns false if the format is not valid for playback with MPD. * This function performs some basic validity checks. */ static inline bool audio_format_valid(const struct audio_format *af) { - return af->sample_rate > 0 && - (af->bits == 8 || af->bits == 16 || af->bits == 24) && - af->channels >= 1; + return audio_valid_sample_rate(af->sample_rate) && + audio_valid_sample_format(af->bits) && + audio_valid_channel_count(af->channels); } static inline bool audio_format_equals(const struct audio_format *a, -- cgit v1.2.3