diff options
author | Max Kellermann <max@duempel.org> | 2014-01-10 21:04:54 +0100 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2014-01-10 22:57:46 +0100 |
commit | 10abb07960071544778bb8abd1456b701834958e (patch) | |
tree | 083a17affe7ad150e4110c76f17dbb429192a8bf /src/db | |
parent | 040a5ddad53ece9803d71972cb33d01a8afb7ad6 (diff) | |
download | mpd-10abb07960071544778bb8abd1456b701834958e.tar.gz mpd-10abb07960071544778bb8abd1456b701834958e.tar.xz mpd-10abb07960071544778bb8abd1456b701834958e.zip |
db/upnp/Directory: make "attributes" a local variable
We only need it inside StartElement(). Reduces bloat.
Diffstat (limited to '')
-rw-r--r-- | src/db/upnp/Directory.cxx | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/src/db/upnp/Directory.cxx b/src/db/upnp/Directory.cxx index 81e01f216..1ce52aa36 100644 --- a/src/db/upnp/Directory.cxx +++ b/src/db/upnp/Directory.cxx @@ -56,7 +56,6 @@ class UPnPDirParser final : public CommonExpatParser { struct StackEl { StackEl(const std::string& nm) : name(nm) {} std::string name; - std::map<std::string,std::string> attributes; }; std::vector<StackEl> m_path; @@ -73,17 +72,18 @@ protected: virtual void StartElement(const XML_Char *name, const XML_Char **attrs) { m_path.push_back(StackEl(name)); - for (int i = 0; attrs[i] != 0; i += 2) { - m_path.back().attributes[attrs[i]] = attrs[i+1]; - } + + std::map<std::string,std::string> attributes; + for (int i = 0; attrs[i] != 0; i += 2) + attributes[attrs[i]] = attrs[i+1]; switch (name[0]) { case 'c': if (!strcmp(name, "container")) { m_tobj.clear(); m_tobj.type = UPnPDirObject::Type::CONTAINER; - m_tobj.m_id = m_path.back().attributes["id"]; - m_tobj.m_pid = m_path.back().attributes["parentID"]; + m_tobj.m_id = attributes["id"]; + m_tobj.m_pid = attributes["parentID"]; } break; @@ -91,8 +91,8 @@ protected: if (!strcmp(name, "item")) { m_tobj.clear(); m_tobj.type = UPnPDirObject::Type::ITEM; - m_tobj.m_id = m_path.back().attributes["id"]; - m_tobj.m_pid = m_path.back().attributes["parentID"]; + m_tobj.m_id = attributes["id"]; + m_tobj.m_pid = attributes["parentID"]; } break; @@ -102,12 +102,12 @@ protected: // bitrate="24576" duration="00:03:35" sampleFrequency="44100" // nrAudioChannels="2"> std::string s; - s="protocolInfo";m_tobj.m_props[s] = m_path.back().attributes[s]; - s="size";m_tobj.m_props[s] = m_path.back().attributes[s]; - s="bitrate";m_tobj.m_props[s] = m_path.back().attributes[s]; - s="duration";m_tobj.m_props[s] = m_path.back().attributes[s]; - s="sampleFrequency";m_tobj.m_props[s] = m_path.back().attributes[s]; - s="nrAudioChannels";m_tobj.m_props[s] = m_path.back().attributes[s]; + s="protocolInfo";m_tobj.m_props[s] = attributes[s]; + s="size";m_tobj.m_props[s] = attributes[s]; + s="bitrate";m_tobj.m_props[s] = attributes[s]; + s="duration";m_tobj.m_props[s] = attributes[s]; + s="sampleFrequency";m_tobj.m_props[s] = attributes[s]; + s="nrAudioChannels";m_tobj.m_props[s] = attributes[s]; } break; |