[ARROW-8353] [C++] is_nullable maybe not initialized in parquet writer

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.17.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/24538
• Assignee: Wes McKinney
• Reporter: Neal Richardson
• Created: 06/Apr/20 21:27
• Updated: 11/Jan/23 07:59
• Resolved: 07/Apr/20 09:04

Description

From the Rtools build:

[ 84%] Building CXX object src/parquet/CMakeFiles/parquet_static.dir/column_reader.cc.obj
In file included from D:/a/arrow/arrow/cpp/src/arrow/io/concurrency.h:23:0,
                 from D:/a/arrow/arrow/cpp/src/arrow/io/memory.h:25,
                 from D:/a/arrow/arrow/cpp/src/parquet/platform.h:25,
                 from D:/a/arrow/arrow/cpp/src/parquet/arrow/writer.h:23,
                 from D:/a/arrow/arrow/cpp/src/parquet/arrow/writer.cc:18:
D:/a/arrow/arrow/cpp/src/arrow/result.h: In member function 'virtual arrow::Status parquet::arrow::FileWriterImpl::WriteColumnChunk(const std::shared_ptr<arrow::ChunkedArray>&, int64_t, int64_t)':
D:/a/arrow/arrow/cpp/src/arrow/result.h:428:28: warning: 'is_nullable' may be used uninitialized in this function [-Wmaybe-uninitialized]
   auto result_name = (rexpr);                               \
                            ^
D:/a/arrow/arrow/cpp/src/parquet/arrow/writer.cc:430:10: note: 'is_nullable' was declared here
     bool is_nullable;
          ^

I'd give it a default value, but IDK that it's that simple.

Issue Links

• links to: GitHub Pull Request #6853