Changeset 10917 for code/branches/cpp11_v2/src/modules/notifications

Timestamp:

Dec 2, 2015, 11:32:08 PM (9 years ago)

Author:

landauf

Message:

made mapEntry const& wherever possible

Location:

code/branches/cpp11_v2/src/modules/notifications

Files:

• NotificationManager.cc (modified) (4 diffs)
• NotificationQueue.cc (modified) (1 diff)

code/branches/cpp11_v2/src/modules/notifications/NotificationManager.cc

@@ -69 +69 @@
     {
         // Destroys all Notifications.
-        for(auto& mapEntry : this->allNotificationsList_)
+        for(const auto& mapEntry : this->allNotificationsList_)
             mapEntry.second->destroy();
         this->allNotificationsList_.clear();
@@ -152 +152 @@
         bool executed = false;
         // Clear all NotificationQueues that have the input sender as target.
-        for(auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
+        for(const auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
         {
             const std::set<std::string>& set = mapEntry.second->getTargetsSet();
@@ -187 +187 @@
 
         // Insert the Notification in all NotificationQueues that have its sender as target.
-        for(auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
+        for(const auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
         {
             const std::set<std::string>& set = mapEntry.second->getTargetsSet();
@@ -345 +345 @@
 
         // Iterate through all Notifications to determine whether any of them should belong to the newly registered NotificationQueue.
-        for(auto& mapEntry : this->allNotificationsList_)
+        for(const auto& mapEntry : this->allNotificationsList_)
         {
             if(!bAll && set.find(mapEntry.second->getSender()) != set.end()) // Checks whether the listener has the sender of the current Notification as target.

code/branches/cpp11_v2/src/modules/notifications/NotificationQueue.cc

@@ -206 +206 @@
         {
             // Add all Notifications that have been created after this NotificationQueue was created.
-            for(auto& mapEntry : *notifications)
+            for(const auto& mapEntry : *notifications)
             {
                 if(mapEntry.first >= this->creationTime_)