From 3f72e2c3b7ce5a83287dc136c68b7ef6d2cc66bd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Remko=20Tron=C3=A7on?= <git@el-tramo.be> Date: Mon, 23 May 2011 20:57:05 +0200 Subject: Fixed const references to vectors. This could potentially cause crashes (same as a previous patch). diff --git a/Swift/Controllers/Roster/RosterController.cpp b/Swift/Controllers/Roster/RosterController.cpp index 706f50a..5b61abf 100644 --- a/Swift/Controllers/Roster/RosterController.cpp +++ b/Swift/Controllers/Roster/RosterController.cpp @@ -142,7 +142,7 @@ void RosterController::handleOnJIDRemoved(const JID& jid) { roster_->removeContact(jid); } -void RosterController::handleOnJIDUpdated(const JID& jid, const std::string& oldName, const std::vector<std::string> passedOldGroups) { +void RosterController::handleOnJIDUpdated(const JID& jid, const std::string& oldName, const std::vector<std::string>& passedOldGroups) { if (oldName != xmppRoster_->getNameForJID(jid)) { roster_->applyOnItems(SetName(nickResolver_->jidToNick(jid), jid)); } diff --git a/Swift/Controllers/Roster/RosterController.h b/Swift/Controllers/Roster/RosterController.h index f224180..0a2b818 100644 --- a/Swift/Controllers/Roster/RosterController.h +++ b/Swift/Controllers/Roster/RosterController.h @@ -57,7 +57,7 @@ namespace Swift { void handleOnJIDAdded(const JID &jid); void handleRosterCleared(); void handleOnJIDRemoved(const JID &jid); - void handleOnJIDUpdated(const JID &jid, const std::string& oldName, const std::vector<std::string> oldGroups); + void handleOnJIDUpdated(const JID &jid, const std::string& oldName, const std::vector<std::string>& oldGroups); void handleStartChatRequest(const JID& contact); void handleChangeStatusRequest(StatusShow::Type show, const std::string &statusText); void handleShowOfflineToggled(bool state); diff --git a/Swiften/Elements/DiscoInfo.h b/Swiften/Elements/DiscoInfo.h index d5bf64a..b73165e 100644 --- a/Swiften/Elements/DiscoInfo.h +++ b/Swiften/Elements/DiscoInfo.h @@ -90,7 +90,7 @@ namespace Swift { extensions_.push_back(form); } - const std::vector<Form::ref> getExtensions() const { + const std::vector<Form::ref>& getExtensions() const { return extensions_; } diff --git a/Swiften/Elements/FormField.h b/Swiften/Elements/FormField.h index f455303..517369b 100644 --- a/Swiften/Elements/FormField.h +++ b/Swiften/Elements/FormField.h @@ -48,7 +48,7 @@ namespace Swift { return options; } - const std::vector<std::string> getRawValues() const { + const std::vector<std::string>& getRawValues() const { return rawValues; } diff --git a/Swiften/Elements/JinglePayload.h b/Swiften/Elements/JinglePayload.h index 59d3c99..59fba7b 100644 --- a/Swiften/Elements/JinglePayload.h +++ b/Swiften/Elements/JinglePayload.h @@ -102,7 +102,7 @@ namespace Swift { this->contents.push_back(content); } - const std::vector<JingleContent::ref> getContents() const { + const std::vector<JingleContent::ref>& getContents() const { return contents; } diff --git a/Swiften/Elements/MUCUserPayload.h b/Swiften/Elements/MUCUserPayload.h index fb6d4c4..7460c35 100644 --- a/Swiften/Elements/MUCUserPayload.h +++ b/Swiften/Elements/MUCUserPayload.h @@ -52,9 +52,9 @@ namespace Swift { void addStatusCode(StatusCode code) {statusCodes_.push_back(code);} - const std::vector<Item> getItems() const {return items_;} + const std::vector<Item>& getItems() const {return items_;} - const std::vector<StatusCode> getStatusCodes() const {return statusCodes_;} + const std::vector<StatusCode>& getStatusCodes() const {return statusCodes_;} private: std::vector<Item> items_; diff --git a/Swiften/Elements/SearchPayload.h b/Swiften/Elements/SearchPayload.h index 3a484cc..d6d7ed1 100644 --- a/Swiften/Elements/SearchPayload.h +++ b/Swiften/Elements/SearchPayload.h @@ -74,7 +74,7 @@ namespace Swift { this->email = v; } - const std::vector<Item> getItems() const { + const std::vector<Item>& getItems() const { return items; } diff --git a/Swiften/Jingle/JingleSession.h b/Swiften/Jingle/JingleSession.h index c00492d..fe8aa01 100644 --- a/Swiften/Jingle/JingleSession.h +++ b/Swiften/Jingle/JingleSession.h @@ -37,7 +37,7 @@ namespace Swift { return JingleContent::ref(); } - const std::vector<JingleContent::ref> getContents() const { + const std::vector<JingleContent::ref>& getContents() const { return contents; } -- cgit v0.10.2-6-g49f6