From: Arnaud Giersch <arnaud.giersch@univ-fcomte.fr>
Date: Sun, 11 Oct 2020 20:58:23 +0000 (+0200)
Subject: More const.
X-Git-Tag: v3.26~331
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/0348239c5ae3b76bd1390ab388ac37e27039e91b

More const.
---

diff --git a/src/instr/instr_config.cpp b/src/instr/instr_config.cpp
index f1c9d17394..a1be1875e4 100644
--- a/src/instr/instr_config.cpp
+++ b/src/instr/instr_config.cpp
@@ -23,7 +23,7 @@ XBT_LOG_NEW_CATEGORY(instr, "Logging the behavior of the tracing system (used fo
 XBT_LOG_NEW_DEFAULT_SUBCATEGORY (instr_config, instr, "Configuration");
 
 std::ofstream tracing_file;
-std::map<container_t, std::ofstream*> tracing_files; // TI specific
+std::map<const simgrid::instr::Container*, std::ofstream*> tracing_files; // TI specific
 double prefix = 0.0;                                 // TI specific
 
 constexpr char OPT_TRACING_BASIC[]             = "tracing/basic";
@@ -216,14 +216,14 @@ int trace_precision;
 /*************
  * Callbacks *
  *************/
-xbt::signal<void(Container&)> Container::on_creation;
-xbt::signal<void(Container&)> Container::on_destruction;
-xbt::signal<void(Type&, e_event_type)> Type::on_creation;
-xbt::signal<void(LinkType&, Type&, Type&)> LinkType::on_creation;
+xbt::signal<void(Container const&)> Container::on_creation;
+xbt::signal<void(Container const&)> Container::on_destruction;
+xbt::signal<void(Type const&, e_event_type)> Type::on_creation;
+xbt::signal<void(LinkType const&, Type const&, Type const&)> LinkType::on_creation;
 xbt::signal<void(PajeEvent&)> PajeEvent::on_creation;
-xbt::signal<void(PajeEvent&)> PajeEvent::on_destruction;
-xbt::signal<void(StateEvent&)> StateEvent::on_destruction;
-xbt::signal<void(EntityValue&)> EntityValue::on_creation;
+xbt::signal<void(PajeEvent const&)> PajeEvent::on_destruction;
+xbt::signal<void(StateEvent const&)> StateEvent::on_destruction;
+xbt::signal<void(EntityValue const&)> EntityValue::on_creation;
 
 static void on_container_creation_paje(const Container& c)
 {
@@ -260,7 +260,7 @@ static void on_container_destruction_paje(const Container& c)
   }
 }
 
-static void on_container_creation_ti(Container& c)
+static void on_container_creation_ti(const Container& c)
 {
   XBT_DEBUG("%s: event_type=%u, timestamp=%f", __func__, PAJE_CreateContainer, SIMIX_get_clock());
   // if we are in the mode with only one file
@@ -286,7 +286,7 @@ static void on_container_creation_ti(Container& c)
   tracing_files.insert({&c, ti_unique_file});
 }
 
-static void on_container_destruction_ti(Container& c)
+static void on_container_destruction_ti(const Container& c)
 {
   if (not trace_disable_destroy && &c != Container::get_root()) {
     if (not simgrid::config::get_value<bool>("tracing/smpi/format/ti-one-file") || tracing_files.size() == 1) {
diff --git a/src/instr/instr_paje_containers.hpp b/src/instr/instr_paje_containers.hpp
index ec2930d622..b342f77b35 100644
--- a/src/instr/instr_paje_containers.hpp
+++ b/src/instr/instr_paje_containers.hpp
@@ -21,8 +21,8 @@ class Container {
   std::string name_; /* Unique name of this container */
 
 public:
-  static xbt::signal<void(Container&)> on_creation;
-  static xbt::signal<void(Container&)> on_destruction;
+  static xbt::signal<void(Container const&)> on_creation;
+  static xbt::signal<void(Container const&)> on_destruction;
 
   explicit Container(const std::string& name, const std::string& type_name, Container* father);
   Container(const Container&) = delete;
diff --git a/src/instr/instr_paje_events.hpp b/src/instr/instr_paje_events.hpp
index 293a2cd0e9..74e63799d5 100644
--- a/src/instr/instr_paje_events.hpp
+++ b/src/instr/instr_paje_events.hpp
@@ -42,7 +42,7 @@ class PajeEvent {
   Type* type_;
 public:
   static xbt::signal<void(PajeEvent&)> on_creation;
-  static xbt::signal<void(PajeEvent&)> on_destruction;
+  static xbt::signal<void(PajeEvent const&)> on_destruction;
 
   double timestamp_;
   e_event_type eventType_;
@@ -78,7 +78,7 @@ class StateEvent : public PajeEvent {
   std::unique_ptr<TIData> extra_;
 
 public:
-  static xbt::signal<void(StateEvent&)> on_destruction;
+  static xbt::signal<void(StateEvent const&)> on_destruction;
   StateEvent(Container* container, Type* type, e_event_type event_type, EntityValue* value, TIData* extra);
   ~StateEvent() override { on_destruction(*this); }
   bool has_extra() const { return extra_ != nullptr; }
diff --git a/src/instr/instr_paje_types.hpp b/src/instr/instr_paje_types.hpp
index b1f8befcd2..27b8b95fc5 100644
--- a/src/instr/instr_paje_types.hpp
+++ b/src/instr/instr_paje_types.hpp
@@ -28,7 +28,7 @@ protected:
   Container* get_issuer() const { return issuer_; }
 
 public:
-  static xbt::signal<void(Type&, e_event_type event_type)> on_creation;
+  static xbt::signal<void(Type const&, e_event_type event_type)> on_creation;
 
   Type(e_event_type event_type, const std::string& name, const std::string& alias, const std::string& color,
        Type* father);
@@ -93,7 +93,7 @@ public:
 
 class LinkType : public ValueType {
 public:
-  static xbt::signal<void(LinkType&, Type&, Type&)> on_creation;
+  static xbt::signal<void(LinkType const&, Type const&, Type const&)> on_creation;
   LinkType(const std::string& name, Type* source, Type* dest, const std::string& alias, Type* father)
       : ValueType(PAJE_DefineLinkType, name, alias, father)
   {
diff --git a/src/instr/instr_paje_values.hpp b/src/instr/instr_paje_values.hpp
index cec347fed1..1fbcc4816f 100644
--- a/src/instr/instr_paje_values.hpp
+++ b/src/instr/instr_paje_values.hpp
@@ -19,7 +19,7 @@ class EntityValue {
   Type* father_;
 
 public:
-  static xbt::signal<void(EntityValue&)> on_creation;
+  static xbt::signal<void(const EntityValue&)> on_creation;
   explicit EntityValue(const std::string& name, const std::string& color, Type* father);
 
   long long int get_id() const { return id_; }