From 3fddc31da8c881aaf62d7a5b6e9888e522d60799 Mon Sep 17 00:00:00 2001
From: Yuya Nishihara <yuya@tcha.org>
Date: Mon, 6 Nov 2023 17:32:28 +0900
Subject: [PATCH] merge: remove Merge::take() which is no longer used

Merge::take() is no longer a cheap function. We can add into_vec() if needed.
---
 lib/src/merge.rs | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/lib/src/merge.rs b/lib/src/merge.rs
index 87eaed8eb..cc7d3aca3 100644
--- a/lib/src/merge.rs
+++ b/lib/src/merge.rs
@@ -165,19 +165,6 @@ impl<T> Merge<T> {
         Merge::new(removes, adds)
     }
 
-    /// Returns the removes and adds as a pair.
-    pub fn take(self) -> (Vec<T>, Vec<T>) {
-        let mut removes = Vec::with_capacity(self.values.len() / 2);
-        let mut adds = Vec::with_capacity(self.values.len() / 2 + 1);
-        let mut values = self.values.into_iter();
-        adds.push(values.next().unwrap());
-        while let Some(remove) = values.next() {
-            removes.push(remove);
-            adds.push(values.next().unwrap());
-        }
-        (removes, adds)
-    }
-
     /// The removed values, also called negative terms.
     pub fn removes(&self) -> impl ExactSizeIterator<Item = &T> {
         self.values[1..].iter().step_by(2)