Fix compiler errors

mathworks · Jul 10, 2023 · ebfa76c · ebfa76c
1 parent d0b79e8
commit ebfa76c
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 27 deletions.
diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/array.cc b/matlab/src/cpp/arrow/matlab/array/proxy/array.cc
@@ -72,25 +72,4 @@ namespace arrow::matlab::array::proxy {
         context.outputs[0] = valid_elements_mda;
     }
 
-    void Array::createDataTypeProxy(libmexclass::proxy::method::Context& context) {
-        const auto type = array->type();
-
-        const auto type_id = array->type_id();
-
-        if (array->null_count() == 0) {
-            ::matlab::data::ArrayFactory factory;
-            auto validity_buffer = factory.createBuffer<bool>(array_length);
-            auto validity_buffer_ptr = validity_buffer.get();
-            std::fill(validity_buffer_ptr, validity_buffer_ptr + array_length, true);
-            auto valid_elements_mda = factory.createArrayFromBuffer<bool>({array_length, 1}, std::move(validity_buffer));
-            context.outputs[0] = valid_elements_mda;
-            return;
-        }
-
-        auto validity_bitmap = array->null_bitmap();
-        auto valid_elements_mda = bit::unpack(validity_bitmap, array_length);
-        context.outputs[0] = valid_elements_mda;
-
-    }
-
 }
diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/array.h b/matlab/src/cpp/arrow/matlab/array/proxy/array.h
@@ -18,9 +18,12 @@
 #pragma once
 
 #include "arrow/array.h"
+#include "arrow/matlab/type/proxy/type.h"
 
 #include "libmexclass/proxy/Proxy.h"
 
+
+
 namespace arrow::matlab::array::proxy {
 
 class Array : public libmexclass::proxy::Proxy {
@@ -43,7 +46,6 @@ class Array : public libmexclass::proxy::Proxy {
 
         virtual void createDataTypeProxy(libmexclass::proxy::method::Context& context) = 0;
 
-        std::shared_ptr<arrow::Array> array;
 };
 
 }
diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/boolean_array.cc b/matlab/src/cpp/arrow/matlab/array/proxy/boolean_array.cc
@@ -67,6 +67,6 @@ namespace arrow::matlab::array::proxy {
         auto boolean_type_proxy = std::make_shared<BooleanTypeProxy>(boolean_type);
         const auto id = libmexclass::proxy::ProxyManager::manageProxy(boolean_type_proxy);
         context.outputs[0] = factory.createScalar(id);
-        context.outputs[1] = factory.createScalar(type->id());
+        context.outputs[1] = factory.createScalar(static_cast<int64_t>(type->id()));
     }
 }
diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/numeric_array.h b/matlab/src/cpp/arrow/matlab/array/proxy/numeric_array.h
@@ -118,7 +118,7 @@ class NumericArray : public arrow::matlab::array::proxy::Array {
             auto numeric_type_proxy = std::make_shared<NumericTypeProxy>(numeric_type);
             const auto id = libmexclass::proxy::ProxyManager::manageProxy(numeric_type_proxy);
             context.outputs[0] = factory.createScalar(id);
-            context.outputs[1] = factory.createScalar(type->id());
+            context.outputs[1] = factory.createScalar(static_cast<int64_t>(type->id()));
         }
 
 };

diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc b/matlab/src/cpp/arrow/matlab/array/proxy/string_array.cc
@@ -92,8 +92,7 @@ namespace arrow::matlab::array::proxy {
         auto string_type_proxy = std::make_shared<StringTypeProxy>(string_type);
         const auto id = libmexclass::proxy::ProxyManager::manageProxy(string_type_proxy);
         context.outputs[0] = factory.createScalar(id);
-        context.outputs[1] = factory.createScalar(type->id());
-
+        context.outputs[1] = factory.createScalar(static_cast<int64_t>(type->id()));
     }
 
 }
diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/timestamp_array.cc b/matlab/src/cpp/arrow/matlab/array/proxy/timestamp_array.cc
@@ -106,6 +106,6 @@ namespace arrow::matlab::array::proxy {
         auto timestamp_type_proxy = std::make_shared<TimestampTypeProxy>(timestamp_type);
         const auto id = libmexclass::proxy::ProxyManager::manageProxy(timestamp_type_proxy);
         context.outputs[0] = factory.createScalar(id);
-        context.outputs[1] = factory.createScalar(type->id());
+        context.outputs[1] = factory.createScalar(static_cast<int64_t>(type->id()));
     }
 }