You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Keith Kraus (Jira)" <ji...@apache.org> on 2021/08/11 04:15:00 UTC
[jira] [Created] (ARROW-13601) [C++] Tests maybe uninitialized
compiler warnings
Keith Kraus created ARROW-13601:
-----------------------------------
Summary: [C++] Tests maybe uninitialized compiler warnings
Key: ARROW-13601
URL: https://issues.apache.org/jira/browse/ARROW-13601
Project: Apache Arrow
Issue Type: Bug
Components: C++
Reporter: Keith Kraus
Using gcc9 I get the following:
{code}
/home/keith/git/arrow/cpp/src/arrow/testing/json_internal.cc: In function 'arrow::Status arrow::testing::json::{anonymous}::GetField(const Value&, arrow::ipc::internal::FieldPosition, arrow::ipc::DictionaryMemo*, std::shared_ptr<arrow::Field>*)':
/home/keith/git/arrow/cpp/src/arrow/testing/json_internal.cc:1128:31: warning: 'is_ordered' may be used uninitialized in this function [-Wmaybe-uninitialized]
1128 | type = ::arrow::dictionary(index_type, type, is_ordered);
| ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
{code}
{code}
In file included from /home/keith/git/arrow/cpp/src/arrow/util/bit_run_reader.h:26,
from /home/keith/git/arrow/cpp/src/arrow/util/bit_util_test.cc:45:
/home/keith/git/arrow/cpp/src/arrow/util/bitmap_reader.h: In member function 'void arrow::TestBitmapUInt64Reader::AssertWords(const arrow::Buffer&, int64_t, int64_t, const std::vector<long unsigned int>&)':
/home/keith/git/arrow/cpp/src/arrow/util/bitmap_reader.h:99:16: warning: 'reader.arrow::internal::BitmapUInt64Reader::carry_bits_' may be used uninitialized in this function [-Wmaybe-uninitialized]
99 | uint64_t word = carry_bits_ | (next_word << num_carry_bits_);
| ^~~~
/home/keith/git/arrow/cpp/src/arrow/util/bit_util_test.cc:242:34: note: 'reader.arrow::internal::BitmapUInt64Reader::carry_bits_' was declared here
242 | internal::BitmapUInt64Reader reader(buffer.data(), start_offset, length);
{code}
{code}
In file included from /home/keith/git/arrow/cpp/src/arrow/util/async_generator.h:30,
from /home/keith/git/arrow/cpp/src/arrow/util/iterator_test.cc:30:
/home/keith/git/arrow/cpp/src/arrow/util/iterator.h: In member function 'arrow::Result<Control> arrow::TransformIterator<T, V>::Next() [with T = std::shared_ptr<int>; V = std::shared_ptr<int>]':
/home/keith/git/arrow/cpp/src/arrow/util/iterator.h:288:12: warning: '*((void*)(& next)+24).std::__shared_count<>::_M_pi' may be used uninitialized in this function [-Wmaybe-uninitialized]
288 | auto next = *next_res;
| ^~~~
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/util/iterator.h:288:12: warning: '*((void*)(& next)+16).std::__shared_ptr<int, __gnu_cxx::_S_atomic>::_M_ptr' may be used uninitialized in this function [-Wmaybe-uninitialized]
[359/671] Building CXX object src/arrow/util/CMakeFiles/arrow-utility-test.dir/utf8_util_test.cc.o
/home/keith/git/arrow/cpp/src/arrow/util/utf8_util_test.cc: In function 'arrow::util::UTF8FindIf_Basics_Test::TestBody()::<lambda(const string&, uint32_t, int64_t, int64_t)>':
/home/keith/git/arrow/cpp/src/arrow/util/utf8_util_test.cc:463:35: warning: 'right' may be used uninitialized in this function [-Wmaybe-uninitialized]
463 | EXPECT_EQ(offset_right, right - begin);
| ^
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/util/utf8_util_test.cc:462:33: warning: 'left' may be used uninitialized in this function [-Wmaybe-uninitialized]
462 | EXPECT_EQ(offset_left, left - begin);
| ^
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/util/utf8_util_test.cc: In function 'arrow::util::UTF8FindIf_Basics_Test::TestBody()::<lambda(const string&, unsigned char, int64_t, int64_t)>':
/home/keith/git/arrow/cpp/src/arrow/util/utf8_util_test.cc:447:35: warning: 'right' may be used uninitialized in this function [-Wmaybe-uninitialized]
447 | EXPECT_EQ(offset_right, right - begin);
| ^
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/util/utf8_util_test.cc:446:33: warning: 'left' may be used uninitialized in this function [-Wmaybe-uninitialized]
446 | EXPECT_EQ(offset_left, left - begin);
| ^
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/util/variant_test.cc: In member function 'virtual void arrow::util::{anonymous}::VariantTest_Visit_Test::TestBody()':
/home/keith/git/arrow/cpp/src/arrow/util/variant_test.cc:260:3: warning: '*((void*)&<anonymous> +8)' may be used uninitialized in this function [-Wmaybe-uninitialized]
260 | for (auto Assert :
| ^~~
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/util/variant_test.cc:230:8: warning: '<anonymous>.arrow::util::{anonymous}::AssertVisitOne<arrow::util::Variant<std::vector<int, std::allocator<int> >, const char*>, std::vector<int> >::expected_' may be used uninitialized in this function [-Wmaybe-uninitialized]
230 | struct AssertVisitOne {
| ^~~~~~~~~~~~~~
{code}
{code}
/home/keith/git/arrow/cpp/src/arrow/gpu/cuda_benchmark.cc: In function 'void arrow::cuda::CudaBufferWriterBenchmark(benchmark::State&, int64_t, int64_t, int64_t)':
/home/keith/git/arrow/cpp/src/arrow/gpu/cuda_benchmark.cc:42:35: warning: 'manager' may be used uninitialized in this function [-Wmaybe-uninitialized]
42 | ABORT_NOT_OK(manager->GetContext(kGpuNumber).Value(&context));
| ^
{code}
{code}
/home/keith/git/arrow/cpp/src/parquet/arrow/arrow_schema_test.cc: In function 'arrow::Result<std::deque<parquet::internal::LevelInfo> > parquet::arrow::RootToTreeLeafLevels(const parquet::arrow::SchemaManifest&, int)':
/home/keith/git/arrow/cpp/src/parquet/arrow/arrow_schema_test.cc:1343:16: warning: 'field' may be used uninitialized in this function [-Wmaybe-uninitialized]
1343 | while (field != nullptr) {
| ~~~~~~^~~~~~~~~~
{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)