From 588a0f6fae335916482f0f95f575a5d1d6b8a28d Mon Sep 17 00:00:00 2001 From: Sarah Gilmore Date: Thu, 6 Jul 2023 09:23:56 -0400 Subject: [PATCH] Use GetView() instead of GetString() --- matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc b/matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc index cc7971bba8ab2..51f39d72fca6c 100644 --- a/matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc +++ b/matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc @@ -66,7 +66,7 @@ namespace arrow::matlab::array::proxy { strings.reserve(array_length); for (size_t i = 0; i < array_length; ++i) { auto string_array = std::static_pointer_cast(array); - const auto& str_utf8 = string_array->GetString(i); + auto str_utf8 = string_array->GetView(i); MATLAB_ASSIGN_OR_ERROR_WITH_CONTEXT(auto str_utf16, arrow::util::UTF8StringToUTF16(str_utf8), context, error::UNICODE_CONVERSION_ERROR_ID); const mda::MATLABString matlab_string = mda::MATLABString(std::move(str_utf16)); strings.push_back(matlab_string);