Seems like this could be fixed by adding default values:

struct DayMilliseconds {
  int32_t days = 0;
  int32_t milliseconds = 0;
  ...
};

In the meantime, you would have to suppress the warning in the
compiler where it's happening

On Tue, Jul 27, 2021 at 12:26 PM Rares Vernica <rvern...@gmail.com> wrote:
>
> Hello,
>
> I'm getting a handful of warnings when including arrow/builder.h Is this
> expected? Should I use the suggested -W flag?
>
> In file included from
> /opt/apache-arrow/include/arrow/array/builder_dict.h:29:0,
>                  from /opt/apache-arrow/include/arrow/builder.h:26,
>
> /opt/apache-arrow/include/arrow/array/builder_primitive.h: In instantiation
> of 'arrow::Status arrow::NumericBuilder<TypeClass>::AppendNull() [with T =
> arrow::DayTimeIntervalType]':
> XArray.cpp:658:1:   required from here
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:102:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::days'
> [-Wmissing-field-initializers]
>      data_builder_.UnsafeAppend(value_type{});  // zero
>      ^
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:102:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::milliseconds'
> [-Wmissing-field-initializers]
> /opt/apache-arrow/include/arrow/array/builder_primitive.h: In instantiation
> of 'arrow::Status arrow::NumericBuilder<TypeClass>::AppendNulls(int64_t)
> [with T = arrow::DayTimeIntervalType; int64_t = long int]':
> XArray.cpp:658:1:   required from here
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:94:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::days'
> [-Wmissing-field-initializers]
>      data_builder_.UnsafeAppend(length, value_type{});  // zero
>      ^
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:94:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::milliseconds'
> [-Wmissing-field-initializers]
> /opt/apache-arrow/include/arrow/array/builder_primitive.h: In instantiation
> of 'arrow::Status arrow::NumericBuilder<TypeClass>::AppendEmptyValue()
> [with T = arrow::DayTimeIntervalType]':
> XArray.cpp:658:1:   required from here
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:110:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::days'
> [-Wmissing-field-initializers]
>      data_builder_.UnsafeAppend(value_type{});  // zero
>      ^
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:110:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::milliseconds'
> [-Wmissing-field-initializers]
> /opt/apache-arrow/include/arrow/array/builder_primitive.h: In instantiation
> of 'arrow::Status
> arrow::NumericBuilder<TypeClass>::AppendEmptyValues(int64_t) [with T =
> arrow::DayTimeIntervalType; int64_t = long int]':
>
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:118:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::days'
> [-Wmissing-field-initializers]
>      data_builder_.UnsafeAppend(length, value_type{});  // zero
>      ^
> /opt/apache-arrow/include/arrow/array/builder_primitive.h:118:5: warning:
> missing initializer for member
> 'arrow::DayTimeIntervalType::DayMilliseconds::milliseconds'
> [-Wmissing-field-initializers]
>
> Thanks!
> Rares

Reply via email to