rtpsw opened a new issue, #36694:
URL: https://github.com/apache/arrow/issues/36694

   ### Describe the enhancement requested
   
   When building a recent clone of Arrow, I observed this warning that should 
be cleaned up:
   ```
   [483/689] Building CXX object 
src/arrow/engine/CMakeFiles/arrow-substrait-substrait-test.dir/substrait/serde_test.cc.o
   In file included from /build/arrow/cpp/src/arrow/datum.h:29,
                    from /build/arrow/cpp/src/arrow/compute/expression.h:29,
                    from /build/arrow/cpp/src/arrow/compute/exec.h:33,
                    from /build/arrow/cpp/src/arrow/compute/kernel.h:32,
                    from /build/arrow/cpp/src/arrow/compute/function.h:27,
                    from /build/arrow/cpp/src/arrow/compute/api_aggregate.h:25,
                    from /build/arrow/cpp/src/arrow/acero/options.h:28,
                    from /build/arrow/cpp/src/arrow/acero/asof_join_node.h:20,
                    from 
/build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc:31:
   In constructor 
‘arrow::BaseBinaryScalar::BaseBinaryScalar(arrow::BaseBinaryScalar&&)’,
       inlined from ‘arrow::BinaryScalar::BinaryScalar(arrow::BinaryScalar&&)’ 
at /build/arrow/cpp/src/arrow/scalar.h:273:21,
       inlined from ‘void __gnu_cxx::new_allocator<_Tp>::construct(_Up*, 
_Args&& ...) [with _Up = arrow::BinaryScalar; _Args = {arrow::BinaryScalar}; 
_Tp = arrow::BinaryScalar]’ at /usr/include/c++/11/ext/new_allocator.h:162:4,
       inlined from ‘static void std::allocator_traits<std::allocator<_Tp1> 
>::construct(std::allocator_traits<std::allocator<_Tp1> >::allocator_type&, 
_Up*, _Args&& ...) [with _Up = arrow::BinaryScalar; _Args = 
{arrow::BinaryScalar}; _Tp = arrow::BinaryScalar]’ at 
/usr/include/c++/11/bits/alloc_traits.h:516:17,
       inlined from ‘std::_Sp_counted_ptr_inplace<_Tp, _Alloc, 
_Lp>::_Sp_counted_ptr_inplace(_Alloc, _Args&& ...) [with _Args = 
{arrow::BinaryScalar}; _Tp = arrow::BinaryScalar; _Alloc = 
std::allocator<arrow::BinaryScalar>; __gnu_cxx::_Lock_policy _Lp = 
__gnu_cxx::_S_atomic]’ at /usr/include/c++/11/bits/shared_ptr_base.h:519:39,
       inlined from ‘std::__shared_count<_Lp>::__shared_count(_Tp*&, 
std::_Sp_alloc_shared_tag<_Alloc>, _Args&& ...) [with _Tp = 
arrow::BinaryScalar; _Alloc = std::allocator<arrow::BinaryScalar>; _Args = 
{arrow::BinaryScalar}; __gnu_cxx::_Lock_policy _Lp = __gnu_cxx::_S_atomic]’ at 
/usr/include/c++/11/bits/shared_ptr_base.h:650:16,
       inlined from ‘std::__shared_ptr<_Tp, 
_Lp>::__shared_ptr(std::_Sp_alloc_shared_tag<_Tp>, _Args&& ...) [with _Alloc = 
std::allocator<arrow::BinaryScalar>; _Args = {arrow::BinaryScalar}; _Tp = 
arrow::BinaryScalar; __gnu_cxx::_Lock_policy _Lp = __gnu_cxx::_S_atomic]’ at 
/usr/include/c++/11/bits/shared_ptr_base.h:1342:14,
       inlined from 
‘std::shared_ptr<_Tp>::shared_ptr(std::_Sp_alloc_shared_tag<_Tp>, _Args&& ...) 
[with _Alloc = std::allocator<arrow::BinaryScalar>; _Args = 
{arrow::BinaryScalar}; _Tp = arrow::BinaryScalar]’ at 
/usr/include/c++/11/bits/shared_ptr.h:409:59,
       inlined from ‘std::shared_ptr<_Tp> std::allocate_shared(const _Alloc&, 
_Args&& ...) [with _Tp = arrow::BinaryScalar; _Alloc = 
std::allocator<arrow::BinaryScalar>; _Args = {arrow::BinaryScalar}]’ at 
/usr/include/c++/11/bits/shared_ptr.h:863:39,
       inlined from ‘std::shared_ptr<_Tp> std::make_shared(_Args&& ...) [with 
_Tp = arrow::BinaryScalar; _Args = {arrow::BinaryScalar}]’ at 
/usr/include/c++/11/bits/shared_ptr.h:879:42,
       inlined from ‘arrow::Datum::Datum(T&&) [with T = arrow::BinaryScalar; TV 
= arrow::BinaryScalar; bool IsArray = false; bool IsScalar = true; 
<template-parameter-1-5> = void]’ at /build/arrow/cpp/src/arrow/datum.h:128:59,
       inlined from ‘virtual void 
arrow::engine::Substrait_SupportedLiterals_Test::TestBody()’ at 
/build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc:491:11:
   /build/arrow/cpp/src/arrow/scalar.h:248:21: warning: 
‘<unnamed>.arrow::BinaryScalar::<unnamed>.arrow::BaseBinaryScalar::<anonymous>’ 
may be used uninitialized [-Wmaybe-uninitialized]
     248 | struct ARROW_EXPORT BaseBinaryScalar
         |                     ^~~~~~~~~~~~~~~~
   /build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc: In member 
function ‘virtual void 
arrow::engine::Substrait_SupportedLiterals_Test::TestBody()’:
   /build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc:491:75: note: 
‘<anonymous>’ declared here
     491 |   ExpectEq(R"({"binary": "enp6"})", 
BinaryScalar(Buffer::FromString("zzz")));
         |                                                                      
     ^
   In file included from /build/arrow/cpp/src/arrow/datum.h:29,
                    from /build/arrow/cpp/src/arrow/compute/expression.h:29,
                    from /build/arrow/cpp/src/arrow/compute/exec.h:33,
                    from /build/arrow/cpp/src/arrow/compute/kernel.h:32,
                    from /build/arrow/cpp/src/arrow/compute/function.h:27,
                    from /build/arrow/cpp/src/arrow/compute/api_aggregate.h:25,
                    from /build/arrow/cpp/src/arrow/acero/options.h:28,
                    from /build/arrow/cpp/src/arrow/acero/asof_join_node.h:20,
                    from 
/build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc:31:
   In constructor 
‘arrow::BaseBinaryScalar::BaseBinaryScalar(arrow::BaseBinaryScalar&&)’,
       inlined from ‘arrow::BinaryScalar::BinaryScalar(arrow::BinaryScalar&&)’ 
at /build/arrow/cpp/src/arrow/scalar.h:273:21,
       inlined from ‘void __gnu_cxx::new_allocator<_Tp>::construct(_Up*, 
_Args&& ...) [with _Up = arrow::BinaryScalar; _Args = {arrow::BinaryScalar}; 
_Tp = arrow::BinaryScalar]’ at /usr/include/c++/11/ext/new_allocator.h:162:4,
       inlined from ‘static void std::allocator_traits<std::allocator<_Tp1> 
>::construct(std::allocator_traits<std::allocator<_Tp1> >::allocator_type&, 
_Up*, _Args&& ...) [with _Up = arrow::BinaryScalar; _Args = 
{arrow::BinaryScalar}; _Tp = arrow::BinaryScalar]’ at 
/usr/include/c++/11/bits/alloc_traits.h:516:17,
       inlined from ‘std::_Sp_counted_ptr_inplace<_Tp, _Alloc, 
_Lp>::_Sp_counted_ptr_inplace(_Alloc, _Args&& ...) [with _Args = 
{arrow::BinaryScalar}; _Tp = arrow::BinaryScalar; _Alloc = 
std::allocator<arrow::BinaryScalar>; __gnu_cxx::_Lock_policy _Lp = 
__gnu_cxx::_S_atomic]’ at /usr/include/c++/11/bits/shared_ptr_base.h:519:39,
       inlined from ‘std::__shared_count<_Lp>::__shared_count(_Tp*&, 
std::_Sp_alloc_shared_tag<_Alloc>, _Args&& ...) [with _Tp = 
arrow::BinaryScalar; _Alloc = std::allocator<arrow::BinaryScalar>; _Args = 
{arrow::BinaryScalar}; __gnu_cxx::_Lock_policy _Lp = __gnu_cxx::_S_atomic]’ at 
/usr/include/c++/11/bits/shared_ptr_base.h:650:16,
       inlined from ‘std::__shared_ptr<_Tp, 
_Lp>::__shared_ptr(std::_Sp_alloc_shared_tag<_Tp>, _Args&& ...) [with _Alloc = 
std::allocator<arrow::BinaryScalar>; _Args = {arrow::BinaryScalar}; _Tp = 
arrow::BinaryScalar; __gnu_cxx::_Lock_policy _Lp = __gnu_cxx::_S_atomic]’ at 
/usr/include/c++/11/bits/shared_ptr_base.h:1342:14,
       inlined from 
‘std::shared_ptr<_Tp>::shared_ptr(std::_Sp_alloc_shared_tag<_Tp>, _Args&& ...) 
[with _Alloc = std::allocator<arrow::BinaryScalar>; _Args = 
{arrow::BinaryScalar}; _Tp = arrow::BinaryScalar]’ at 
/usr/include/c++/11/bits/shared_ptr.h:409:59,
       inlined from ‘std::shared_ptr<_Tp> std::allocate_shared(const _Alloc&, 
_Args&& ...) [with _Tp = arrow::BinaryScalar; _Alloc = 
std::allocator<arrow::BinaryScalar>; _Args = {arrow::BinaryScalar}]’ at 
/usr/include/c++/11/bits/shared_ptr.h:863:39,
       inlined from ‘std::shared_ptr<_Tp> std::make_shared(_Args&& ...) [with 
_Tp = arrow::BinaryScalar; _Args = {arrow::BinaryScalar}]’ at 
/usr/include/c++/11/bits/shared_ptr.h:879:42,
       inlined from ‘arrow::Datum::Datum(T&&) [with T = arrow::BinaryScalar; TV 
= arrow::BinaryScalar; bool IsArray = false; bool IsScalar = true; 
<template-parameter-1-5> = void]’ at /build/arrow/cpp/src/arrow/datum.h:128:59,
       inlined from ‘virtual void 
arrow::engine::Substrait_SupportedLiterals_Test::TestBody()’ at 
/build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc:491:11:
   /build/arrow/cpp/src/arrow/scalar.h:248:21: warning: 
‘<unnamed>.arrow::BinaryScalar::<unnamed>.arrow::BaseBinaryScalar::<anonymous>’ 
may be used uninitialized [-Wmaybe-uninitialized]
     248 | struct ARROW_EXPORT BaseBinaryScalar
         |                     ^~~~~~~~~~~~~~~~
   /build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc: In member 
function ‘virtual void 
arrow::engine::Substrait_SupportedLiterals_Test::TestBody()’:
   /build/arrow/cpp/src/arrow/engine/substrait/serde_test.cc:491:75: note: 
‘<anonymous>’ declared here
     491 |   ExpectEq(R"({"binary": "enp6"})", 
BinaryScalar(Buffer::FromString("zzz")));
         |                                                                      
     ^
   ```
   
   ### Component(s)
   
   C++


-- 
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]

Reply via email to