Merge pull request #4311 from bska/dont-needlessly-copy-name-vectors

Don't Needlessly Copy Well/Group Names
OPM · Nov 5, 2024 · c0f2302 · c0f2302
2 parents 9dcad9a + 63d7733
commit c0f2302
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/opm/input/eclipse/Schedule/UDQ/UDQContext.cpp b/opm/input/eclipse/Schedule/UDQ/UDQContext.cpp
@@ -207,7 +207,7 @@ namespace Opm {
         return it->second;
     }
 
-    std::vector<std::string> UDQContext::wells() const
+    const std::vector<std::string>& UDQContext::wells() const
     {
         return this->well_matcher.wells();
     }
@@ -217,7 +217,7 @@ namespace Opm {
         return this->well_matcher.wells(pattern);
     }
 
-    std::vector<std::string> UDQContext::groups() const
+    const std::vector<std::string>& UDQContext::groups() const
     {
         return this->summary_state.groups();
     }

diff --git a/opm/input/eclipse/Schedule/UDQ/UDQContext.hpp b/opm/input/eclipse/Schedule/UDQ/UDQContext.hpp
@@ -89,9 +89,9 @@ namespace Opm {
 
         const UDQFunctionTable& function_table() const;
 
-        std::vector<std::string> wells() const;
+        const std::vector<std::string>& wells() const;
         std::vector<std::string> wells(const std::string& pattern) const;
-        std::vector<std::string> groups() const;
+        const std::vector<std::string>& groups() const;
         SegmentSet segments() const;
         SegmentSet segments(const std::vector<std::string>& set_descriptor) const;