X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/96cedde3cdbc0b8ffc3f096a1b65d021b0226f99..84402e8e2ee2a2d0bef25fdceb0a263ed8b471f6:/src/instr/instr_paje_containers.hpp

diff --git a/src/instr/instr_paje_containers.hpp b/src/instr/instr_paje_containers.hpp
index a2d5216b91..4a76b0cb6d 100644
--- a/src/instr/instr_paje_containers.hpp
+++ b/src/instr/instr_paje_containers.hpp
@@ -1,4 +1,4 @@
-/* Copyright (c) 2010-2019. The SimGrid Team. All rights reserved.          */
+/* Copyright (c) 2010-2020. The SimGrid Team. All rights reserved.          */
 
 /* This program is free software; you can redistribute it and/or modify it
  * under the terms of the license (GNU LGPL) which comes with this package. */
@@ -20,7 +20,9 @@ class Container {
   long long int id_;
   std::string name_; /* Unique name of this container */
 public:
-  Container(std::string name, std::string type_name, Container* father);
+  Container(const std::string& name, const std::string& type_name, Container* father);
+  Container(const Container&) = delete;
+  Container& operator=(const Container&) = delete;
   virtual ~Container();
 
   Type* type_; /* Type of this container */
@@ -28,35 +30,35 @@ public:
   std::map<std::string, Container*> children_;
   kernel::routing::NetPoint* netpoint_ = nullptr;
 
-  static Container* by_name_or_null(std::string name);
-  static Container* by_name(std::string name);
-  std::string get_name() { return name_; }
+  static Container* by_name_or_null(const std::string& name);
+  static Container* by_name(const std::string& name);
+  const std::string& get_name() const { return name_; }
   const char* get_cname() { return name_.c_str(); }
   long long int get_id() { return id_; }
   void remove_from_parent();
   void log_creation();
   void log_destruction();
 
-  StateType* get_state(std::string name);
-  LinkType* get_link(std::string name);
-  VariableType* get_variable(std::string name);
-  void create_child(std::string name, std::string type_name);
+  StateType* get_state(const std::string& name);
+  LinkType* get_link(const std::string& name);
+  VariableType* get_variable(const std::string& name);
+  void create_child(const std::string& name, const std::string& type_name);
   static Container* get_root();
 };
 
 class NetZoneContainer : public Container {
 public:
-  NetZoneContainer(std::string name, unsigned int level, NetZoneContainer* father);
+  NetZoneContainer(const std::string& name, unsigned int level, NetZoneContainer* father);
 };
 
 class RouterContainer : public Container {
 public:
-  RouterContainer(std::string name, Container* father);
+  RouterContainer(const std::string& name, Container* father);
 };
 
 class HostContainer : public Container {
 public:
-  HostContainer(simgrid::s4u::Host& host, NetZoneContainer* father);
+  HostContainer(simgrid::s4u::Host const& host, NetZoneContainer* father);
 };
 }
 }