nealrichardson commented on pull request #8650:
URL: https://github.com/apache/arrow/pull/8650#issuecomment-740195890
FYI Windows build failure is
```
r_to_arrow.cpp: In instantiation of 'arrow::Status
arrow::r::VisitInt64Vector(SEXP, R_xlen_t, VisitorFunc&&) [with VisitorFunc =
arrow::r::VisitVector(SEXP, R_xlen_t, T*) [with T =
arrow::internal::Converter<arrow::r::RScalar*, arrow::r::RConversionOptions>;
SEXP = SEXPREC*; R_xlen_t = int]::<lambda(arrow::r::RScalar*)>; SEXP =
SEXPREC*; R_xlen_t = int]':
r_to_arrow.cpp:512:84: required from 'arrow::Status
arrow::r::VisitVector(SEXP, R_xlen_t, T*) [with T =
arrow::internal::Converter<arrow::r::RScalar*, arrow::r::RConversionOptions>;
SEXP = SEXPREC*; R_xlen_t = int]'
r_to_arrow.cpp:535:40: required from 'arrow::Status arrow::r::Extend(T*,
SEXP, R_xlen_t) [with T = arrow::internal::Converter<arrow::r::RScalar*,
arrow::r::RConversionOptions>; SEXP = SEXPREC*; R_xlen_t = int]'
r_to_arrow.cpp:762:7: required from here
r_to_arrow.cpp:451:66: error: dereferencing type-punned pointer will break
strict-aliasing rules [-Werror=strict-aliasing]
obj.null = is_NA<int64_t>(*reinterpret_cast<int64_t*>(&value));
^
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]