Remove redundant constructor

cpcloud · cpcloud · commit fe13ef3615df · 2017-08-24T09:21:56.000-04:00
diff --git a/cpp/src/arrow/array.cc b/cpp/src/arrow/array.cc
@@ -348,18 +348,8 @@ const uint8_t* FixedSizeBinaryArray::GetValue(int64_t i) const {
 DecimalArray::DecimalArray(const std::shared_ptr<internal::ArrayData>& data)
     : FixedSizeBinaryArray(data) {
   DCHECK_EQ(data->type->id(), Type::DECIMAL);
-  SetData(data);
 }
 
-DecimalArray::DecimalArray(const std::shared_ptr<DataType>& type, int64_t length,
-                           const std::shared_ptr<Buffer>& data,
-                           const std::shared_ptr<Buffer>& null_bitmap, int64_t null_count,
-                           int64_t offset)
-    : FixedSizeBinaryArray(type, length, data, null_bitmap, null_count, offset) {
-  BufferVector buffers = {null_bitmap, data};
-  SetData(
-      std::make_shared<ArrayData>(type, length, std::move(buffers), null_count, offset));
-}
 
 #define DECIMAL_TO_STRING_CASE(bits, bytes, precision, scale) \
   case bits: {                                                \
diff --git a/cpp/src/arrow/array.h b/cpp/src/arrow/array.h
@@ -538,14 +538,11 @@ class ARROW_EXPORT DecimalArray : public FixedSizeBinaryArray {
  public:
   using TypeClass = DecimalType;
 
+  using FixedSizeBinaryArray::FixedSizeBinaryArray;
+
   /// \brief Construct DecimalArray from internal::ArrayData instance
   explicit DecimalArray(const std::shared_ptr<internal::ArrayData>& data);
 
-  DecimalArray(const std::shared_ptr<DataType>& type, int64_t length,
-               const std::shared_ptr<Buffer>& data,
-               const std::shared_ptr<Buffer>& null_bitmap = nullptr,
-               int64_t null_count = 0, int64_t offset = 0);
-
   std::string FormatValue(int64_t i) const;
 
   std::shared_ptr<Array> Slice(int64_t offset, int64_t length) const override;
