From 9e9418294a69bfcb27b8cc631ad32b1e6507f3c6 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Fri, 15 Mar 2019 20:28:27 +0100 Subject: [PATCH] song/TagSongFilter: eliminate Match(TagItem) --- src/song/TagSongFilter.cxx | 10 ++-------- src/song/TagSongFilter.hxx | 1 - 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/src/song/TagSongFilter.cxx b/src/song/TagSongFilter.cxx index 4b587c920..df92ed435 100644 --- a/src/song/TagSongFilter.cxx +++ b/src/song/TagSongFilter.cxx @@ -34,13 +34,6 @@ TagSongFilter::ToExpression() const noexcept + " \"" + EscapeFilterString(filter.GetValue()) + "\")"; } -bool -TagSongFilter::Match(const TagItem &item) const noexcept -{ - return (type == TAG_NUM_OF_ITEM_TYPES || item.type == type) && - filter.Match(item.value); -} - bool TagSongFilter::Match(const Tag &tag) const noexcept { @@ -49,7 +42,8 @@ TagSongFilter::Match(const Tag &tag) const noexcept for (const auto &i : tag) { visited_types[i.type] = true; - if (Match(i)) + if ((type == TAG_NUM_OF_ITEM_TYPES || i.type == type) && + filter.Match(i.value)) return true; } diff --git a/src/song/TagSongFilter.hxx b/src/song/TagSongFilter.hxx index 6d0449d61..3f9f7051b 100644 --- a/src/song/TagSongFilter.hxx +++ b/src/song/TagSongFilter.hxx @@ -69,7 +69,6 @@ public: private: bool Match(const Tag &tag) const noexcept; - bool Match(const TagItem &tag) const noexcept; }; #endif