felipecrv opened a new pull request, #38265:
URL: https://github.com/apache/arrow/pull/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
