remove uncessary clone in verify_queue

tx-pool/src/component/tests/chunk.rs

@@ -57,6 +57,15 @@ async fn verify_queue_basic() {
     let counts = count.await.unwrap();
     assert_eq!(counts, vec![1, 1, 1, 2]);
 
+    let cur = queue.pop_first();
+    assert_eq!(cur.unwrap().tx, tx);
+
+    assert!(!queue.is_empty());
+    let cur = queue.pop_first();
+    assert_eq!(cur.unwrap().tx, tx2);
+
+    assert!(queue.is_empty());
+
     queue.clear();
     assert!(!queue.contains_key(&id));
 }

tx-pool/src/component/verify_queue.rs

@@ -103,20 +103,19 @@ impl VerifyQueue {
 
     /// Returns the first entry in the queue and remove it
     pub fn pop_first(&mut self) -> Option<Entry> {
-        if let Some(entry) = self.get_first() {
-            self.remove_tx(&entry.tx.proposal_short_id());
-            Some(entry)
+        if let Some(short_id) = self.peak_first() {
+            self.remove_tx(&short_id)
         } else {
             None
         }
     }
 
     /// Returns the first entry in the queue
-    pub fn get_first(&self) -> Option<Entry> {
+    pub fn peak_first(&self) -> Option<ProposalShortId> {
         self.inner
             .iter_by_added_time()
             .next()
-            .map(|entry| entry.inner.clone())
+            .map(|entry| entry.inner.tx.proposal_short_id())
     }
 
     /// If the queue did not have this tx present, true is returned.

tx-pool/src/verify_mgr.rs

@@ -77,7 +77,7 @@ impl Worker {
             return;
         }
 
-        if self.tasks.read().await.get_first().is_none() {
+        if self.tasks.read().await.peak_first().is_none() {
             return;
         }
         // pick a entry to run verify