Commit b4c517c5 authored by Max Kellermann's avatar Max Kellermann

song/AudioFormatFilter: add mask support

parent b39bc85e
......@@ -278,6 +278,14 @@
<listitem>
<para>
"<code>(AudioFormat =~ 'SAMPLERATE:BITS:CHANNELS')</code>":
matches the audio format with the given mask (i.e. one
or more attributes may be "<code>*</code>").
</para>
</listitem>
<listitem>
<para>
"<code>(!EXPRESSION)</code>": negate an expression.
</para>
</listitem>
......
......@@ -24,13 +24,14 @@
std::string
AudioFormatSongFilter::ToExpression() const noexcept
{
// TODO: support mask
return std::string("(AudioFormat == \"") + ToString(value).c_str() + "\")";
return std::string("(AudioFormat ") +
(value.IsFullyDefined() ? "==" : "=~") +
" \"" + ToString(value).c_str() + "\")";
}
bool
AudioFormatSongFilter::Match(const LightSong &song) const noexcept
{
// TODO: support mask
return song.audio_format == value;
return song.audio_format.IsDefined() &&
song.audio_format.MatchMask(value);
}
......@@ -245,14 +245,18 @@ SongFilter::ParseExpression(const char *&s, bool fold_case)
return std::make_unique<BaseSongFilter>(std::move(value));
} else if (type == LOCATE_TAG_AUDIO_FORMAT) {
if (s[0] != '=' || s[1] != '=')
throw std::runtime_error("'==' expected");
bool mask;
if (s[0] == '=' && s[1] == '=')
mask = false;
else if (s[0] == '=' && s[1] == '~')
mask = true;
else
throw std::runtime_error("'==' or '=~' expected");
s = StripLeft(s + 2);
// TODO: support mask
const auto value = ParseAudioFormat(ExpectQuoted(s).c_str(),
false);
mask);
if (*s != ')')
throw std::runtime_error("')' expected");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment