Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Allow for callback disconnection
authorFrederic Suter <frederic.suter@cc.in2p3.fr>
Fri, 19 Apr 2019 11:35:00 +0000 (13:35 +0200)
committerFrederic Suter <frederic.suter@cc.in2p3.fr>
Fri, 19 Apr 2019 11:35:00 +0000 (13:35 +0200)
xbt::signal::connect now returns an unsigned int id and callbacks are
stored in a map <id, callback>.
a new xbt::signal::disconnect(unsigned int id) method has been added.

include/xbt/signal.hpp

index 2900ad4..a5e6544 100644 (file)
@@ -7,8 +7,8 @@
 #define SIMGRID_XBT_SIGNAL_HPP
 
 #include <functional>
 #define SIMGRID_XBT_SIGNAL_HPP
 
 #include <functional>
+#include <map>
 #include <utility>
 #include <utility>
-#include <vector>
 
 namespace simgrid {
 namespace xbt {
 
 namespace simgrid {
 namespace xbt {
@@ -23,20 +23,22 @@ namespace xbt {
   */
   template<class R, class... P>
   class signal<R(P...)> {
   */
   template<class R, class... P>
   class signal<R(P...)> {
-  private:
     typedef std::function<R(P...)> callback_type;
     typedef std::function<R(P...)> callback_type;
-    std::vector<callback_type> handlers_;
+    std::map<unsigned int, callback_type> handlers_;
+    unsigned int callback_sequence_id = 0;
+
   public:
   public:
-    template<class U>
-    void connect(U slot)
+    template <class U> unsigned int connect(U slot)
     {
     {
-      handlers_.push_back(std::move(slot));
+      handlers_.insert({callback_sequence_id, std::move(slot)});
+      return callback_sequence_id++;
     }
     R operator()(P... args) const
     {
       for (auto const& handler : handlers_)
     }
     R operator()(P... args) const
     {
       for (auto const& handler : handlers_)
-        handler(args...);
+        handler.second(args...);
     }
     }
+    void disconnect(unsigned int id) { handlers_.erase(id); }
     void disconnect_slots() { handlers_.clear(); }
     int get_slot_count() { return handlers_.size(); }
   };
     void disconnect_slots() { handlers_.clear(); }
     int get_slot_count() { return handlers_.size(); }
   };