[arrow] branch main updated: GH-38263 [C++]: Prefer to call string_view::data() instead of begin() where a char pointer is expected (#38265)

[raulcd]

This is an automated email from the ASF dual-hosted git repository.

raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 93ca3b2fdf GH-38263 [C++]: Prefer to call string_view::data() instead 
of begin() where a char pointer is expected (#38265)
93ca3b2fdf is described below

commit 93ca3b2fdfb813542b14759c4d918553500bc307
Author: Felipe Oliveira Carvalho <felipe...@gmail.com>
AuthorDate: Mon Oct 16 06:05:57 2023 -0300

    GH-38263 [C++]: Prefer to call string_view::data() instead of begin() where 
a char pointer is expected (#38265)
    
    ### Rationale for this change
    
    The MSVC compiler doesn't seem to allow user code to assume 
`std::string_view::const_iterator` is `const char*`, so using only 
`re2::StringPiece` and preferring to call `.data()` instead of `.begin()` 
should make things more uniform across different compilers and STL 
implementations.
    
    ### What changes are included in this PR?
    
     - Using `re2::StringPiece` instead of `std::string_view` to interact with 
`re2`
     - Use `data()` instead of `begin()` where a `char*` is expected
    
    ### Are these changes tested?
    
    Yes, by existing tests.
    * Closes: #38263
    
    Authored-by: Felipe Oliveira Carvalho <felipe...@gmail.com>
    Signed-off-by: Raúl Cumplido <raulcumpl...@gmail.com>
---
 cpp/src/arrow/compute/kernels/scalar_string_ascii.cc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/cpp/src/arrow/compute/kernels/scalar_string_ascii.cc 
b/cpp/src/arrow/compute/kernels/scalar_string_ascii.cc
index a90a7282bb..90f5d40b98 100644
--- a/cpp/src/arrow/compute/kernels/scalar_string_ascii.cc
+++ b/cpp/src/arrow/compute/kernels/scalar_string_ascii.cc
@@ -2063,12 +2063,12 @@ struct RegexSubstringReplacer {
         regex_find_("(" + options_.pattern + ")", MakeRE2Options<Type>()),
         regex_replacement_(options_.pattern, MakeRE2Options<Type>()) {}
 
-  Status ReplaceString(std::string_view s, TypedBufferBuilder<uint8_t>* 
builder) const {
+  Status ReplaceString(re2::StringPiece s, TypedBufferBuilder<uint8_t>* 
builder) const {
     re2::StringPiece replacement(options_.replacement);
 
     // If s is empty, then it's essentially global
     if (options_.max_replacements == -1 || s.empty()) {
-      std::string s_copy(s);
+      std::string s_copy{s.data(), s.length()};
       RE2::GlobalReplace(&s_copy, regex_replacement_, replacement);
       return builder->Append(reinterpret_cast<const uint8_t*>(s_copy.data()),
                              s_copy.length());
@@ -2079,18 +2079,18 @@ struct RegexSubstringReplacer {
     // We might do this faster similar to RE2::GlobalReplace using Match and 
Rewrite
     const char* i = s.data();
     const char* end = s.data() + s.length();
-    re2::StringPiece piece(s.data(), s.length());
+    re2::StringPiece mutable_s{s};
 
     int64_t max_replacements = options_.max_replacements;
     while ((i < end) && (max_replacements != 0)) {
       std::string found;
-      if (!RE2::FindAndConsume(&piece, regex_find_, &found)) {
+      if (!RE2::FindAndConsume(&mutable_s, regex_find_, &found)) {
         RETURN_NOT_OK(builder->Append(reinterpret_cast<const uint8_t*>(i),
                                       static_cast<int64_t>(end - i)));
         i = end;
       } else {
         // wind back to the beginning of the match
-        const char* pos = piece.begin() - found.length();
+        const char* pos = mutable_s.data() - found.length();
         // the string before the pattern
         RETURN_NOT_OK(builder->Append(reinterpret_cast<const uint8_t*>(i),
                                       static_cast<int64_t>(pos - i)));
@@ -2101,7 +2101,7 @@ struct RegexSubstringReplacer {
         RETURN_NOT_OK(builder->Append(reinterpret_cast<const 
uint8_t*>(found.data()),
                                       static_cast<int64_t>(found.length())));
         // skip pattern
-        i = piece.begin();
+        i = mutable_s.data();
         max_replacements--;
       }
     }