diff options
author | Max Kellermann <max@duempel.org> | 2014-01-21 23:05:13 +0100 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2014-01-21 23:07:33 +0100 |
commit | 154250f551468c58116e2d558726022d2cd9a199 (patch) | |
tree | 531b98666fce9706e34da4250b79d698a981c724 /src/db | |
parent | 3e4fb92cfb90eb147fb57f76bb33f9c59dc43585 (diff) | |
download | mpd-154250f551468c58116e2d558726022d2cd9a199.tar.gz mpd-154250f551468c58116e2d558726022d2cd9a199.tar.xz mpd-154250f551468c58116e2d558726022d2cd9a199.zip |
db/upnp: eliminate redundant Namei() parameter "oobjid"
The caller should better obtain the object id from the returned
UPnPDirObject instance.
Diffstat (limited to '')
-rw-r--r-- | src/db/UpnpDatabasePlugin.cxx | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/src/db/UpnpDatabasePlugin.cxx b/src/db/UpnpDatabasePlugin.cxx index 7b2f4996b..cd3b8afbd 100644 --- a/src/db/UpnpDatabasePlugin.cxx +++ b/src/db/UpnpDatabasePlugin.cxx @@ -130,7 +130,7 @@ private: bool Namei(ContentDirectoryService &server, const std::vector<std::string> &vpath, - std::string &oobjid, UPnPDirObject &dirent, + UPnPDirObject &dirent, Error &error) const; /** @@ -232,8 +232,7 @@ UpnpDatabase::GetSong(const char *uri, Error &error) const vpath.erase(vpath.begin()); UPnPDirObject dirent; if (vpath.front() != rootid) { - std::string objid; - if (!Namei(server, vpath, objid, dirent, error)) + if (!Namei(server, vpath, dirent, error)) return nullptr; } else { if (!ReadNode(server, vpath.back().c_str(), dirent, @@ -465,10 +464,9 @@ UpnpDatabase::BuildPath(ContentDirectoryService &server, bool UpnpDatabase::Namei(ContentDirectoryService &server, const std::vector<std::string> &vpath, - std::string &oobjid, UPnPDirObject &odirent, + UPnPDirObject &odirent, Error &error) const { - oobjid.clear(); std::string objid(rootid); if (vpath.empty()) { @@ -476,7 +474,6 @@ UpnpDatabase::Namei(ContentDirectoryService &server, if (!ReadNode(server, rootid, odirent, error)) return false; - oobjid = rootid; return true; } @@ -503,7 +500,6 @@ UpnpDatabase::Namei(ContentDirectoryService &server, if (i == vpath.size() - 1) { // The last element in the path was found and it's // a container, we're done - oobjid = objid; odirent = std::move(*child); return true; } @@ -513,7 +509,6 @@ UpnpDatabase::Namei(ContentDirectoryService &server, // If this is the last path elt, we found the target, // else it does not exist if (i == vpath.size() - 1) { - oobjid = objid; odirent = std::move(*child); return true; } else { @@ -565,9 +560,8 @@ UpnpDatabase::VisitServer(ContentDirectoryService &server, } // Translate the target path into an object id and the associated metadata. - std::string objid; UPnPDirObject tdirent; - if (!Namei(server, vpath, objid, tdirent, error)) + if (!Namei(server, vpath, tdirent, error)) return false; /* If recursive is set, this is a search... No use sending it @@ -575,7 +569,7 @@ UpnpDatabase::VisitServer(ContentDirectoryService &server, recursion (1-deep) here, which will handle the "add dir" case. */ if (selection.recursive && selection.filter) - return SearchSongs(server, objid.c_str(), selection, + return SearchSongs(server, tdirent.m_id.c_str(), selection, visit_song, error); if (tdirent.type == UPnPDirObject::Type::ITEM) { @@ -610,7 +604,8 @@ UpnpDatabase::VisitServer(ContentDirectoryService &server, and loop here, but it's not useful as mpd will only return data to the client when we're done anyway. */ UPnPDirContent dirbuf; - if (!server.readDir(m_lib->getclh(), objid.c_str(), dirbuf, error)) + if (!server.readDir(m_lib->getclh(), tdirent.m_id.c_str(), dirbuf, + error)) return false; for (auto &dirent : dirbuf.objects) { |