1 files changed, 80 insertions, 82 deletions
diff --git a/Swiften/MUC/MUCBookmarkManager.cpp b/Swiften/MUC/MUCBookmarkManager.cpp
index 4eb2ae6..9f8ae77 100644
--- a/Swiften/MUC/MUCBookmarkManager.cpp
+++ b/Swiften/MUC/MUCBookmarkManager.cpp
@@ -1,121 +1,119 @@
 /*
- * Copyright (c) 2010 Isode Limited.
+ * Copyright (c) 2010-2016 Isode Limited.
  * All rights reserved.
  * See the COPYING file for more information.
  */
 
-#include "MUCBookmarkManager.h"
+#include <Swiften/MUC/MUCBookmarkManager.h>
+
+#include <memory>
 
 #include <boost/bind.hpp>
-#include <boost/smart_ptr/make_shared.hpp>
-#include <iostream>
 
-#include <Swiften/Base/foreach.h>
 #include <Swiften/Queries/IQRouter.h>
 #include <Swiften/Queries/Requests/GetPrivateStorageRequest.h>
 #include <Swiften/Queries/Requests/SetPrivateStorageRequest.h>
 
-
 namespace Swift {
 
 MUCBookmarkManager::MUCBookmarkManager(IQRouter* iqRouter) {
-	iqRouter_ = iqRouter;
-	ready_ = false;
-	GetPrivateStorageRequest<Storage>::ref request = GetPrivateStorageRequest<Storage>::create(iqRouter_);
-	request->onResponse.connect(boost::bind(&MUCBookmarkManager::handleBookmarksReceived, this, _1, _2));
-	request->send();
+    iqRouter_ = iqRouter;
+    ready_ = false;
+    GetPrivateStorageRequest<Storage>::ref request = GetPrivateStorageRequest<Storage>::create(iqRouter_);
+    request->onResponse.connect(boost::bind(&MUCBookmarkManager::handleBookmarksReceived, this, _1, _2));
+    request->send();
 }
 
-void MUCBookmarkManager::handleBookmarksReceived(boost::shared_ptr<Storage> payload, ErrorPayload::ref error) {
-	if (error) {
-		return;
-	}
-
-	ready_ = true;
-	onBookmarksReady();
-
-	storage = payload;
-
-	std::vector<MUCBookmark> receivedBookmarks;
-	foreach (Storage::Room room, payload->getRooms()) {
-		receivedBookmarks.push_back(MUCBookmark(room));
-	}
-
-	std::vector<MUCBookmark> newBookmarks;
-	foreach (const MUCBookmark& oldBookmark, bookmarks_) {
-		if (containsEquivalent(receivedBookmarks, oldBookmark)) {
-			newBookmarks.push_back(oldBookmark);
-		} else {
-			onBookmarkRemoved(oldBookmark);
-		} 
-	}
-
-	foreach (const MUCBookmark& newBookmark, receivedBookmarks) {
-		if (!containsEquivalent(bookmarks_, newBookmark)) {
-			newBookmarks.push_back(newBookmark);
-			onBookmarkAdded(newBookmark);
-		}
-	}
-	bookmarks_ = newBookmarks;
+void MUCBookmarkManager::handleBookmarksReceived(std::shared_ptr<Storage> payload, ErrorPayload::ref error) {
+    if (error) {
+        return;
+    }
+
+    ready_ = true;
+    onBookmarksReady();
+
+    storage = payload;
+
+    std::vector<MUCBookmark> receivedBookmarks;
+    for (const auto& room : payload->getRooms()) {
+        receivedBookmarks.push_back(MUCBookmark(room));
+    }
+
+    std::vector<MUCBookmark> newBookmarks;
+    for (const auto& oldBookmark : bookmarks_) {
+        if (containsEquivalent(receivedBookmarks, oldBookmark)) {
+            newBookmarks.push_back(oldBookmark);
+        } else {
+            onBookmarkRemoved(oldBookmark);
+        }
+    }
+
+    for (const auto& newBookmark : receivedBookmarks) {
+        if (!containsEquivalent(bookmarks_, newBookmark)) {
+            newBookmarks.push_back(newBookmark);
+            onBookmarkAdded(newBookmark);
+        }
+    }
+    bookmarks_ = newBookmarks;
 }
 
 bool MUCBookmarkManager::containsEquivalent(const std::vector<MUCBookmark>& list, const MUCBookmark& bookmark) {
-	return std::find(list.begin(), list.end(), bookmark) != list.end();
+    return std::find(list.begin(), list.end(), bookmark) != list.end();
 }
 
 void MUCBookmarkManager::replaceBookmark(const MUCBookmark& oldBookmark, const MUCBookmark& newBookmark) {
-	if (!ready_) return;
-	for (size_t i = 0; i < bookmarks_.size(); i++) {
-		if (bookmarks_[i] == oldBookmark) {
-			bookmarks_[i] = newBookmark;
-			flush();
-			onBookmarkRemoved(oldBookmark);
-			onBookmarkAdded(newBookmark);
-			return;
-		}
-	}
+    if (!ready_) return;
+    for (auto& bookmark : bookmarks_) {
+        if (bookmark == oldBookmark) {
+            bookmark = newBookmark;
+            flush();
+            onBookmarkRemoved(oldBookmark);
+            onBookmarkAdded(newBookmark);
+            return;
+        }
+    }
 }
 
 void MUCBookmarkManager::addBookmark(const MUCBookmark& bookmark) {
-	if (!ready_) return;
-	bookmarks_.push_back(bookmark);
-	onBookmarkAdded(bookmark);
-	flush();
+    if (!ready_) return;
+    bookmarks_.push_back(bookmark);
+    onBookmarkAdded(bookmark);
+    flush();
 }
 
 
 void MUCBookmarkManager::removeBookmark(const MUCBookmark& bookmark) {
-	if (!ready_) return;
-	std::vector<MUCBookmark>::iterator it;
-	for (it = bookmarks_.begin(); it != bookmarks_.end(); ++it) {
-		if ((*it) == bookmark) {
-			bookmarks_.erase(it);
-			onBookmarkRemoved(bookmark);
-			break;
-		}
-	}
-	flush();
+    if (!ready_) return;
+    std::vector<MUCBookmark>::iterator it;
+    for (it = bookmarks_.begin(); it != bookmarks_.end(); ++it) {
+        if ((*it) == bookmark) {
+            bookmarks_.erase(it);
+            onBookmarkRemoved(bookmark);
+            break;
+        }
+    }
+    flush();
 }
 
 void MUCBookmarkManager::flush() {
-	if (!storage) {
-		storage = boost::make_shared<Storage>();
-	}
-	// Update the storage element
-	storage->clearRooms();
-	foreach(const MUCBookmark& bookmark, bookmarks_) {
-		storage->addRoom(bookmark.toStorage());
-	}
-
-	// Send an iq to save the storage element
-	SetPrivateStorageRequest<Storage>::ref request = SetPrivateStorageRequest<Storage>::create(storage, iqRouter_);
-	// FIXME: We should care about the result
-	//request->onResponse.connect(boost::bind(&MUCBookmarkManager::handleBookmarksSet, this, _1, _2));
-	request->send();
+    if (!storage) {
+        storage = std::make_shared<Storage>();
+    }
+    // Update the storage element
+    storage->clearRooms();
+    for (const auto& bookmark : bookmarks_) {
+        storage->addRoom(bookmark.toStorage());
+    }
+
+    // Send an iq to save the storage element
+    SetPrivateStorageRequest<Storage>::ref request = SetPrivateStorageRequest<Storage>::create(storage, iqRouter_);
+    // FIXME: We should care about the result
+    //request->onResponse.connect(boost::bind(&MUCBookmarkManager::handleBookmarksSet, this, _1, _2));
+    request->send();
 }
 
 const std::vector<MUCBookmark>& MUCBookmarkManager::getBookmarks() const {
-	return bookmarks_;
+    return bookmarks_;
 }
 
 }