Constify.

[simgrid.git] / src / instr / instr_paje_types.hpp

diff --git a/src/instr/instr_paje_types.hpp b/src/instr/instr_paje_types.hpp
index 80c7a8b..47ed87c 100644 (file)
--- a/src/instr/instr_paje_types.hpp
+++ b/src/instr/instr_paje_types.hpp
@@ -30,16 +30,16 @@ public:
   Type(std::string name, std::string alias, std::string color, Type* father);
   virtual ~Type();
 
-  std::string get_name() { return name_; }
+  const std::string& get_name() const { return name_; }
   const char* get_cname() { return name_.c_str(); }
   long long int get_id() { return id_; }
   bool is_colored() { return not color_.empty(); }
 
-  Type* by_name(std::string name);
-  LinkType* by_name_or_create(std::string name, Type* source, Type* dest);
-  VariableType* by_name_or_create(std::string name, std::string color);
+  Type* by_name(const std::string& name);
+  LinkType* by_name_or_create(const std::string& name, Type* source, Type* dest);
+  VariableType* by_name_or_create(const std::string& name, std::string color);
 
-  template <class T> T* by_name_or_create(std::string name)
+  template <class T> T* by_name_or_create(const std::string& name)
   {
     auto cont = children_.find(name);
     return cont == children_.end() ? new T(name, this) : static_cast<T*>(cont->second);
@@ -53,14 +53,14 @@ public:
 
 class ContainerType : public Type {
 public:
-  explicit ContainerType(std::string name) : Type(name, name, "", nullptr){};
-  ContainerType(std::string name, Type* father);
+  explicit ContainerType(const std::string& name) : Type(name, name, "", nullptr){};
+  ContainerType(const std::string& name, Type* father);
 };
 
 class VariableType : public Type {
   std::vector<VariableEvent*> events_;
 public:
-  VariableType(std::string name, std::string color, Type* father);
+  VariableType(const std::string& name, std::string color, Type* father);
   ~VariableType();
   void instr_event(double now, double delta, const char* resource, double value);
   void set_event(double timestamp, double value);
@@ -72,11 +72,11 @@ class ValueType : public Type {
 public:
   std::map<std::string, EntityValue*> values_;
   ValueType(std::string name, std::string alias, Type* father) : Type(std::move(name), std::move(alias), "", father){};
-  ValueType(std::string name, Type* father) : Type(name, name, "", father){};
+  ValueType(const std::string& name, Type* father) : Type(name, name, "", father){};
   virtual ~ValueType();
-  void add_entity_value(std::string name, std::string color);
-  void add_entity_value(std::string name);
-  EntityValue* get_entity_value(std::string name);
+  void add_entity_value(const std::string& name, std::string color);
+  void add_entity_value(const std::string& name);
+  EntityValue* get_entity_value(const std::string& name);
 };
 
 class LinkType : public ValueType {
@@ -97,9 +97,9 @@ class StateType : public ValueType {
 public:
   StateType(std::string name, Type* father);
   ~StateType();
-  void set_event(std::string value_name);
-  void push_event(std::string value_name);
-  void push_event(std::string value_name, TIData* extra);
+  void set_event(const std::string& value_name);
+  void push_event(const std::string& value_name);
+  void push_event(const std::string& value_name, TIData* extra);
   void pop_event();
   void pop_event(TIData* extra);
 };