[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [akonadi-calendar] src: Add missing reserve() calls
From:       Sergio Martins <iamsergio () gmail ! com>
Date:       2015-04-30 22:36:40
Message-ID: E1Ynx4S-0003xk-P3 () scm ! kde ! org
[Download RAW message or body]

Git commit 6545327dfba67cf19e6ed5e78cb3702155d102f1 by Sergio Martins.
Committed on 30/04/2015 at 22:36.
Pushed by smartins into branch 'master'.

Add missing reserve() calls

M  +1    -0    src/incidencechanger.cpp
M  +3    -1    src/utils_p.cpp

http://commits.kde.org/akonadi-calendar/6545327dfba67cf19e6ed5e78cb3702155d102f1

diff --git a/src/incidencechanger.cpp b/src/incidencechanger.cpp
index 64ba1bd..8a5a694 100644
--- a/src/incidencechanger.cpp
+++ b/src/incidencechanger.cpp
@@ -362,6 +362,7 @@ void IncidenceChanger::Private::handleDeleteJobResult(KJob *job)
 
     QSharedPointer<DeletionChange> deletionChange = change.staticCast<DeletionChange>();
 
+    deletionChange->mItemIds.reserve(items.count());
     foreach (const Akonadi::Item &item, items) {
         deletionChange->mItemIds.append(item.id());
     }
diff --git a/src/utils_p.cpp b/src/utils_p.cpp
index 4340fc0..979925a 100644
--- a/src/utils_p.cpp
+++ b/src/utils_p.cpp
@@ -106,7 +106,9 @@ bool Akonadi::CalendarUtils::thatIsMe(const QString &_email)
 QStringList Akonadi::CalendarUtils::allEmails()
 {
     QStringList emails;
-    foreach(const QString &email, KIdentityManagement::allEmails()) {
+    const QSet<QString> &allEmails = KIdentityManagement::allEmails();
+    emails.reserve(allEmails.count());
+    foreach(const QString &email, allEmails) {
         emails.append(email);
     }
     return emails;
[prev in list] [next in list] [prev in thread] [next in thread]