Retaining the default copy constructor

Author: MaheshGPai

tdigest/include/tdigest.hpp

@@ -106,12 +106,6 @@ class tdigest {
    */
   explicit tdigest(uint16_t k = DEFAULT_K, const Allocator& allocator = Allocator());
 
-  /**
- * Copy constructor
- * @param other sketch to be copied
- */
-  tdigest(const tdigest& other);
-
   /**
    * Update this t-Digest with the given value
    * @param value to update the t-Digest with

tdigest/include/tdigest_impl.hpp

@@ -597,24 +597,6 @@ bool tdigest<T, A>::is_single_value() const {
   return get_total_weight() == 1;
 }
 
-template<typename T, typename A>
-tdigest<T, A>::tdigest(const tdigest<T, A>& other):
-  reverse_merge_(other.reverse_merge_),
-  k_(other.k_),
-  min_(other.min_),
-  max_(other.max_),
-  centroids_capacity_(other.centroids_capacity_),
-  centroids_(other.centroids_, other.get_allocator()),
-  centroids_weight_(other.centroids_weight_),
-  buffer_(other.buffer_, other.get_allocator())
-{
-  if (other.k_ < 10) throw std::invalid_argument("k must be at least 10");
-  const size_t fudge = other.k_ < 30 ? 30 : 10;
-  centroids_capacity_ = 2 * k_ + fudge;
-  centroids_.reserve(centroids_capacity_);
-  buffer_.reserve(centroids_capacity_ * BUFFER_MULTIPLIER);
-}
-
 template<typename T, typename A>
 tdigest<T, A>::tdigest(bool reverse_merge, uint16_t k, T min, T max, vector_centroid&& centroids, uint64_t weight, vector_t&& buffer):
 reverse_merge_(reverse_merge),