Use reference to const for large input parameters.

diff --git a/include/simgrid/jedule/jedule_platform.hpp b/include/simgrid/jedule/jedule_platform.hpp
index b7232c1..945f561 100644 (file)
--- a/include/simgrid/jedule/jedule_platform.hpp
+++ b/include/simgrid/jedule/jedule_platform.hpp
@@ -35,7 +35,7 @@ public:
   unsigned int get_id_by_name(const char* name) const { return name2id.at(name); }
 
   void add_child(Container* child);
-  void add_resources(std::vector<sg_host_t> hosts);
+  void add_resources(const std::vector<sg_host_t>& hosts);
   void create_hierarchy(const_sg_netzone_t from_as);
   std::vector<int> get_hierarchy();
   std::string get_hierarchy_as_string();

diff --git a/src/instr/jedule/jedule_platform.cpp b/src/instr/jedule/jedule_platform.cpp
index 05a7b40..00f78da 100644 (file)
--- a/src/instr/jedule/jedule_platform.cpp
+++ b/src/instr/jedule/jedule_platform.cpp
@@ -36,7 +36,7 @@ void Container::add_child(jed_container_t child)
   child->set_parent(this);
 }
 
-void Container::add_resources(std::vector<sg_host_t> hosts)
+void Container::add_resources(const std::vector<sg_host_t>& hosts)
 {
   children_.clear();
   last_id_ = 0;
@@ -144,7 +144,7 @@ void Container::print(FILE* jed_file)
 } // namespace jedule
 } // namespace simgrid
 
-static void add_subsets_to(std::vector<simgrid::jedule::Subset>& subset_list, std::vector<const char*> hostgroup,
+static void add_subsets_to(std::vector<simgrid::jedule::Subset>& subset_list, const std::vector<const char*>& hostgroup,
                            jed_container_t parent)
 {
   // get ids for each host