Swiftdiff options
Diffstat (limited to 'Swiften/Avatars/CombinedAvatarProvider.cpp')
| -rw-r--r-- | Swiften/Avatars/CombinedAvatarProvider.cpp | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/Swiften/Avatars/CombinedAvatarProvider.cpp b/Swiften/Avatars/CombinedAvatarProvider.cpp index d283664..1bd74dd 100644 --- a/Swiften/Avatars/CombinedAvatarProvider.cpp +++ b/Swiften/Avatars/CombinedAvatarProvider.cpp @@ -15,3 +15,3 @@ namespace Swift { -std::string CombinedAvatarProvider::getAvatarHash(const JID& jid) const { +boost::optional<std::string> CombinedAvatarProvider::getAvatarHash(const JID& jid) const { return getCombinedAvatarAndCache(jid); @@ -38,3 +38,3 @@ void CombinedAvatarProvider::handleAvatarChanged(const JID& jid) { } - std::string newHash = getCombinedAvatarAndCache(jid); + boost::optional<std::string> newHash = getCombinedAvatarAndCache(jid); if (newHash != oldHash) { @@ -45,6 +45,6 @@ void CombinedAvatarProvider::handleAvatarChanged(const JID& jid) { -std::string CombinedAvatarProvider::getCombinedAvatarAndCache(const JID& jid) const { +boost::optional<std::string> CombinedAvatarProvider::getCombinedAvatarAndCache(const JID& jid) const { SWIFT_LOG(debug) << "JID: " << jid << std::endl; - std::string hash; - for (size_t i = 0; i < providers.size() && hash.empty(); ++i) { + boost::optional<std::string> hash; + for (size_t i = 0; i < providers.size() && !hash; ++i) { hash = providers[i]->getAvatarHash(jid); @@ -52,3 +52,7 @@ std::string CombinedAvatarProvider::getCombinedAvatarAndCache(const JID& jid) co } - avatars[jid] = hash; + if (hash) { + avatars[jid] = *hash; + } else { + avatars[jid] = ""; + } return hash; |