diff options
| author | Max Kellermann <max@duempel.org> | 2014-01-10 08:57:31 +0100 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2014-01-10 08:57:31 +0100 |
| commit | cfc25e08dc84b450aa1296ae031dfaa39178f4ae (patch) | |
| tree | 4c9c6eaf03a3e0a09de3261d49923211f8167fae /src | |
| parent | 1583eb36e46ea5210ac80b60d305e1ca2805ba11 (diff) | |
| download | mpd-cfc25e08dc84b450aa1296ae031dfaa39178f4ae.tar.gz mpd-cfc25e08dc84b450aa1296ae031dfaa39178f4ae.tar.xz mpd-cfc25e08dc84b450aa1296ae031dfaa39178f4ae.zip | |
db/upnp: use TagTable in upnpItemToSong()
Reduces bloat by eliminating one std::map.
Diffstat (limited to '')
| -rw-r--r-- | src/db/UpnpDatabasePlugin.cxx | 39 |
1 files changed, 14 insertions, 25 deletions
diff --git a/src/db/UpnpDatabasePlugin.cxx b/src/db/UpnpDatabasePlugin.cxx index c619316ef..9613e6a8c 100644 --- a/src/db/UpnpDatabasePlugin.cxx +++ b/src/db/UpnpDatabasePlugin.cxx @@ -48,6 +48,17 @@ static const char *const rootid = "0"; +static const struct tag_table upnp_tags[] = { + { "upnp:artist", TAG_ARTIST }, + { "upnp:album", TAG_ALBUM }, + { "upnp:originalTrackNumber", TAG_TRACK }, + { "upnp:genre", TAG_GENRE }, + { "dc:title", TAG_TITLE }, + + /* sentinel */ + { nullptr, TAG_NUM_OF_ITEM_TYPES } +}; + class UpnpDatabase : public Database { LibUPnP *m_lib; UPnPDeviceDirectory *m_superdir; @@ -276,17 +287,6 @@ upnpDurationToSeconds(const std::string &duration) return atoi(v[0].c_str())*3600 + atoi(v[1].c_str())*60 + atoi(v[2].c_str()); } -// Upnp element to mpd tag number correspondance. -// Don't know the upnp equivalent if any: TAG_ALBUM_ARTIST TAG_NAME -// TAG_DATE TAG_COMPOSER TAG_PERFORMER TAG_COMMENT TAG_DISC -static std::map<std::string, TagType> propToTag = { - {"upnp:artist", TAG_ARTIST}, - {"upnp:album", TAG_ALBUM}, - {"upnp:originalTrackNumber", TAG_TRACK}, - {"upnp:genre", TAG_GENRE}, - {"dc:title", TAG_TITLE}, -}; - // If uri is empty, we use the object's url instead. This happens // when the target of a Visit() is a song, which only happens when // "add"ing AFAIK. Visit() calls us with a null uri so that the url @@ -314,9 +314,9 @@ upnpItemToSong(const UPnPDirObject &dirent, const char *uri) tag.AddItem(TAG_TITLE, titleToPathElt(dirent.m_title).c_str()); - for (auto &pt : propToTag) - if (dirent.getprop(pt.first, sprop)) - tag.AddItem(pt.second, sprop.c_str()); + for (auto i = upnp_tags; i->name != nullptr; ++i) + if (dirent.getprop(i->name, sprop)) + tag.AddItem(i->type, sprop.c_str()); s->tag = tag.CommitNew(); return s; @@ -364,17 +364,6 @@ UpnpDatabase::GetSong(const char *uri, Error &error) const return song; } -static const struct tag_table upnp_tags[] = { - { "upnp:artist", TAG_ARTIST }, - { "upnp:album", TAG_ALBUM }, - { "upnp:originalTrackNumber", TAG_TRACK }, - { "upnp:genre", TAG_GENRE }, - { "dc:title", TAG_TITLE }, - - /* sentinel */ - { nullptr, TAG_NUM_OF_ITEM_TYPES } -}; - /** * Retrieve the value for an MPD tag from an object entry. */ |