passing the initalizer list by reference

AsherGlick · Oct 4, 2023 · aede4e4 · aede4e4
1 parent b68be58
commit aede4e4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/xml_converter/src/string_hierarchy.cpp b/xml_converter/src/string_hierarchy.cpp
@@ -37,7 +37,7 @@ bool StringHierarchy::in_hierarchy(
 // ambiguity between the vector and string overloads of the function.
 ////////////////////////////////////////////////////////////////////////////////
 bool StringHierarchy::in_hierarchy(
-    const std::initializer_list<std::string> input) const {
+    const std::initializer_list<std::string> &input) const {
     std::vector<std::string> vec;
     vec.insert(vec.end(), input.begin(), input.end());
     return this->in_hierarchy(vec);
@@ -95,7 +95,7 @@ const StringHierarchy *StringHierarchy::sub_hierarchy(
 // prevent ambiguity between the vector and string overloads of the function.
 ////////////////////////////////////////////////////////////////////////////////
 const StringHierarchy *StringHierarchy::sub_hierarchy(
-    const std::initializer_list<std::string> input) const {
+    const std::initializer_list<std::string> &input) const {
     std::vector<std::string> vec;
     vec.insert(vec.end(), input.begin(), input.end());
     return this->sub_hierarchy(vec);

diff --git a/xml_converter/src/string_hierarchy.hpp b/xml_converter/src/string_hierarchy.hpp
@@ -13,13 +13,13 @@ class StringHierarchy {
         const std::vector<std::string> &path) const;
 
     bool in_hierarchy(
-        const std::initializer_list<std::string>) const;
+        const std::initializer_list<std::string> &input) const;
 
     const StringHierarchy *sub_hierarchy(
         const std::string &node) const;
 
     const StringHierarchy *sub_hierarchy(
-        const std::initializer_list<std::string> input) const;
+        const std::initializer_list<std::string> &input) const;
 
     const StringHierarchy *sub_hierarchy(
         const std::vector<std::string> &path) const;