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

kou 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 66ee781dc2 GH-46986: [CI][C++] Fix a build error with C++20 (#46987)
66ee781dc2 is described below

commit 66ee781dc2c7964950c7066d139fd592ac5700e5
Author: Sutou Kouhei <[email protected]>
AuthorDate: Sat Jul 5 06:44:54 2025 +0900

    GH-46986: [CI][C++] Fix a build error with C++20 (#46987)
    
    ### Rationale for this change
    
    We can't convert `u8"..."` to `std::string` automatically with C++20 
because C++20 uses `std::u8string` for UTF-8 string.
    
    This is related to #46815.
    
    ### What changes are included in this PR?
    
    Remove needless `u8` prefix.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    No.
    * GitHub Issue: #46986
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 cpp/src/arrow/compute/kernels/scalar_string_test.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/compute/kernels/scalar_string_test.cc 
b/cpp/src/arrow/compute/kernels/scalar_string_test.cc
index 4b8e26f3f9..b279f991f6 100644
--- a/cpp/src/arrow/compute/kernels/scalar_string_test.cc
+++ b/cpp/src/arrow/compute/kernels/scalar_string_test.cc
@@ -2271,7 +2271,7 @@ TYPED_TEST(TestStringKernels, UTF8ZeroFill) {
                    R"(["xxx1", "-xx2", "+xx3"])", &options);
 
   // Non-ASCII padding character
-  options = ZeroFillOptions{/*width=*/5, u8"💠"};
+  options = ZeroFillOptions{/*width=*/5, "💠"};
   this->CheckUnary("utf8_zero_fill", R"(["1", "-2", "+3"])", this->type(),
                    R"(["💠💠💠💠1", "-💠💠💠2", "+💠💠💠3"])", &options);
 

Reply via email to