kevingurney commented on code in PR #36615:
URL: https://github.com/apache/arrow/pull/36615#discussion_r1261647536
##########
matlab/src/cpp/arrow/matlab/array/proxy/timestamp_array.cc:
##########
@@ -20,24 +20,22 @@
#include "arrow/matlab/error/error.h"
#include "arrow/matlab/bit/pack.h"
#include "arrow/matlab/bit/unpack.h"
+#include "arrow/matlab/buffer/matlab_buffer.h"
#include "arrow/matlab/type/time_unit.h"
#include "arrow/util/utf8.h"
#include "arrow/type.h"
-#include "arrow/builder.h"
-
namespace arrow::matlab::array::proxy {
- namespace {
- const uint8_t* getUnpackedValidityBitmap(const
::matlab::data::TypedArray<bool>& valid_elements) {
- const auto valid_elements_iterator(valid_elements.cbegin());
- return reinterpret_cast<const
uint8_t*>(valid_elements_iterator.operator->());
- }
- } // anonymous namespace
+ TimestampArray::TimestampArray(std::shared_ptr<arrow::TimestampArray>
timestamp_array)
Review Comment:
Perhaps we should go with the convention of naming these input `Array`
subclass constructor arguments `array` for consistency across the various
subclasses.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
