Take string argument by const&

Unidata · Feb 23, 2022 · e8de071 · e8de071
1 parent 6b1ae26
commit e8de071
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cxx4/ncGroup.cpp b/cxx4/ncGroup.cpp
@@ -1336,7 +1336,7 @@ map<string,NcGroup> NcGroup::getCoordVars(NcGroup::Location location) const {
 }
 
 // Get the NcDim and NcVar object pair for a named coordinate variables.
-void NcGroup::getCoordVar(string& coordVarName, NcDim& ncDim, NcVar& ncVar, NcGroup::Location location) const {
+void NcGroup::getCoordVar(const string& coordVarName, NcDim& ncDim, NcVar& ncVar, NcGroup::Location location) const {
 
   // search in current group and parent groups.
   multimap<string,NcDim>::iterator itD;

diff --git a/cxx4/ncGroup.h b/cxx4/ncGroup.h
@@ -559,7 +559,7 @@ namespace netCDF
       \param location Enumeration type controlling the groups to search.
       \return         The set of names of dimension variables.
     */
-    void getCoordVar(std::string& coordVarName, NcDim& ncDim, NcVar& ncVar, NcGroup::Location location=Current) const;
+    void getCoordVar(const std::string& coordVarName, NcDim& ncDim, NcVar& ncVar, NcGroup::Location location=Current) const;
 
 
   protected: