diff options
author | Max Kellermann <max@duempel.org> | 2012-08-15 21:32:34 +0200 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2012-08-15 23:02:27 +0200 |
commit | a6ac0f89656b9ef374703d24bbb27316a705eadc (patch) | |
tree | 3706b1b8474ae06890595a8e20c8be011f6a162b /src/DatabaseHelpers.cxx | |
parent | 4e1eb03287c1af889372ed4c63220a88d2032f78 (diff) | |
download | mpd-a6ac0f89656b9ef374703d24bbb27316a705eadc.tar.gz mpd-a6ac0f89656b9ef374703d24bbb27316a705eadc.tar.xz mpd-a6ac0f89656b9ef374703d24bbb27316a705eadc.zip |
DatabasePlugin: add method VisitUniqueTags()
Optimize the ProxyDatabase by invoking "list" on the peer, instead of
visiting all songs.
Diffstat (limited to '')
-rw-r--r-- | src/DatabaseHelpers.cxx | 78 |
1 files changed, 78 insertions, 0 deletions
diff --git a/src/DatabaseHelpers.cxx b/src/DatabaseHelpers.cxx new file mode 100644 index 000000000..9a0931137 --- /dev/null +++ b/src/DatabaseHelpers.cxx @@ -0,0 +1,78 @@ +/* + * Copyright (C) 2003-2012 The Music Player Daemon Project + * http://www.musicpd.org + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#include "DatabaseHelpers.hxx" +#include "DatabasePlugin.hxx" +#include "song.h" +#include "tag.h" + +#include <functional> +#include <set> + +#include <string.h> + +struct StringLess { + gcc_pure + bool operator()(const char *a, const char *b) const { + return strcmp(a, b) < 0; + } +}; + +typedef std::set<const char *, StringLess> StringSet; + +static bool +CollectTags(StringSet &set, enum tag_type tag_type, song &song) +{ + struct tag *tag = song.tag; + if (tag == nullptr) + return true; + + bool found = false; + for (unsigned i = 0; i < tag->num_items; ++i) { + if (tag->items[i]->type == tag_type) { + set.insert(tag->items[i]->value); + found = true; + } + } + + if (!found) + set.insert(""); + + return true; +} + +bool +VisitUniqueTags(const Database &db, const DatabaseSelection &selection, + enum tag_type tag_type, + VisitString visit_string, + GError **error_r) +{ + StringSet set; + + using namespace std::placeholders; + const auto f = std::bind(CollectTags, std::ref(set), tag_type, _1); + if (!db.Visit(selection, f, error_r)) + return false; + + for (auto value : set) + if (!visit_string(value, error_r)) + return false; + + return true; +} |