[ARROW-3110] [C++] Compilation warnings with gcc 7.3.0

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.10.0
• Fix Version/s: 0.11.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/19464
• Assignee: Antoine Pitrou
• Reporter: Antoine Pitrou
• Created: 22/Aug/18 17:20
• Updated: 11/Jan/23 07:25
• Resolved: 22/Aug/18 19:04

Description

This is happening when building in release mode:

../src/arrow/python/python_to_arrow.cc: In function 'arrow::Status arrow::py::detail::BuilderAppend(arrow::BinaryBuilder*, PyObject*, bool*)':
../src/arrow/python/python_to_arrow.cc:388:56: warning: 'length' may be used uninitialized in this function [-Wmaybe-uninitialized]
   if (ARROW_PREDICT_FALSE(builder->value_data_length() + length > kBinaryMemoryLimit)) {
                                                        ^
../src/arrow/python/python_to_arrow.cc:385:11: note: 'length' was declared here
   int32_t length;
           ^~~~~~
In file included from ../src/arrow/python/serialize.cc:32:0:
../src/arrow/builder.h: In member function 'arrow::Status arrow::py::SequenceBuilder::Update(int64_t, int8_t*)':
../src/arrow/builder.h:413:5: warning: 'offset32' may be used uninitialized in this function [-Wmaybe-uninitialized]
     raw_data_[length_++] = val;
     ^~~~~~~~~
../src/arrow/python/serialize.cc:90:13: note: 'offset32' was declared here
     int32_t offset32;
             ^~~~~~~~

Issue Links

• links to: GitHub Pull Request #2464