From 9c2cd5273ed0e2e5b20f21c9afcfb0c0b8412ddc Mon Sep 17 00:00:00 2001
From: Jared Duffey <jared.duffey@bluequartz.net>
Date: Tue, 31 Oct 2023 14:26:31 -0400
Subject: [PATCH] Changed to pass string by ref

Signed-off-by: Jared Duffey <jared.duffey@bluequartz.net>
---
 src/complex/Utilities/DataArrayUtilities.cpp | 2 +-
 src/complex/Utilities/DataArrayUtilities.hpp | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/complex/Utilities/DataArrayUtilities.cpp b/src/complex/Utilities/DataArrayUtilities.cpp
index 348a5e82f1..f91081eaad 100644
--- a/src/complex/Utilities/DataArrayUtilities.cpp
+++ b/src/complex/Utilities/DataArrayUtilities.cpp
@@ -315,7 +315,7 @@ std::unique_ptr<MaskCompare> InstantiateMaskCompare(IDataArray& maskArray)
 }
 
 //-----------------------------------------------------------------------------
-bool ConvertIDataArray(const std::shared_ptr<IDataArray>& dataArray, std::string dataFormat)
+bool ConvertIDataArray(const std::shared_ptr<IDataArray>& dataArray, const std::string& dataFormat)
 {
   auto dataType = dataArray->getDataType();
   switch(dataType)
diff --git a/src/complex/Utilities/DataArrayUtilities.hpp b/src/complex/Utilities/DataArrayUtilities.hpp
index cfccd653fd..af2c67e6ac 100644
--- a/src/complex/Utilities/DataArrayUtilities.hpp
+++ b/src/complex/Utilities/DataArrayUtilities.hpp
@@ -470,7 +470,7 @@ Result<> CreateArray(DataStructure& dataStructure, const std::vector<usize>& tup
 }
 
 template <typename T>
-std::shared_ptr<AbstractDataStore<T>> ConvertDataStore(const AbstractDataStore<T>& dataStore, std::string dataFormat)
+std::shared_ptr<AbstractDataStore<T>> ConvertDataStore(const AbstractDataStore<T>& dataStore, const std::string& dataFormat)
 {
   if(dataStore.getDataFormat() == dataFormat)
   {
@@ -489,7 +489,7 @@ std::shared_ptr<AbstractDataStore<T>> ConvertDataStore(const AbstractDataStore<T
 }
 
 template <typename T>
-bool ConvertDataArray(const std::shared_ptr<DataArray<T>> dataArray, std::string dataFormat)
+bool ConvertDataArray(const std::shared_ptr<DataArray<T>> dataArray, const std::string& dataFormat)
 {
   if(dataArray == nullptr)
   {
@@ -506,7 +506,7 @@ bool ConvertDataArray(const std::shared_ptr<DataArray<T>> dataArray, std::string
   return true;
 }
 
-bool ConvertIDataArray(const std::shared_ptr<IDataArray>& dataArray, std::string dataFormat);
+bool ConvertIDataArray(const std::shared_ptr<IDataArray>& dataArray, const std::string& dataFormat);
 
 /**
  * @brief Creates a NeighborList array with the given properties