Searching before insertion is not necessary with std::set.

diff --git a/examples/s4u/app-bittorrent/s4u-tracker.cpp b/examples/s4u/app-bittorrent/s4u-tracker.cpp
index e3eb5e9..8f36504 100644 (file)
--- a/examples/s4u/app-bittorrent/s4u-tracker.cpp
+++ b/examples/s4u/app-bittorrent/s4u-tracker.cpp
@@ -38,9 +38,7 @@ void Tracker::operator()()
       xbt_assert(query != nullptr);
 
       // Add the peer to our peer list, if not already known.
-      if (known_peers.find(query->getPeerId()) == known_peers.end()) {
-        known_peers.insert(query->getPeerId());
-      }
+      known_peers.emplace(query->getPeerId());
 
       // Sending back peers to the requesting peer
       auto* answer = new TrackerAnswer(TRACKER_QUERY_INTERVAL);

diff --git a/src/simdag/sd_global.cpp b/src/simdag/sd_global.cpp
index 71edd95..5a9e822 100644 (file)
--- a/src/simdag/sd_global.cpp
+++ b/src/simdag/sd_global.cpp
@@ -51,8 +51,7 @@ std::set<SD_task_t>* simulate(double how_long){
         SD_task_set_state(task, SD_DONE);
 
         /* the state has changed. Add it only if it's the first change */
-        if (sd_global->return_set.find(task) == sd_global->return_set.end())
-          sd_global->return_set.insert(task);
+        sd_global->return_set.emplace(task);
 
         /* remove the dependencies after this task */
         for (auto const& succ : *task->successors) {