Remove retain_mut crate (#11926)

* Remove retain_mut crate

* Remove reain_mut crate from babe-consensus

substrate/Cargo.lock

@@ -7446,12 +7446,6 @@ dependencies = [
  "quick-error 1.2.3",
 ]
 
-[[package]]
-name = "retain_mut"
-version = "0.1.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "448296241d034b96c11173591deaa1302f2c17b56092106c1f92c1bc0183a8c9"
-
 [[package]]
 name = "rfc6979"
 version = "0.1.0"
@@ -7965,7 +7959,6 @@ dependencies = [
  "parking_lot 0.12.0",
  "rand 0.7.3",
  "rand_chacha 0.2.2",
- "retain_mut",
  "sc-block-builder",
  "sc-client-api",
  "sc-consensus",
@@ -8877,7 +8870,6 @@ dependencies = [
  "parity-scale-codec",
  "parity-util-mem",
  "parking_lot 0.12.0",
- "retain_mut",
  "sc-block-builder",
  "sc-client-api",
  "sc-transaction-pool-api",

substrate/client/consensus/babe/Cargo.toml

@@ -26,7 +26,6 @@ num-rational = "0.2.2"
 num-traits = "0.2.8"
 parking_lot = "0.12.0"
 rand = "0.7.2"
-retain_mut = "0.1.4"
 schnorrkel = { version = "0.9.1", features = ["preaudit_deprecated"] }
 serde = { version = "1.0.136", features = ["derive"] }
 thiserror = "1.0"

substrate/client/consensus/babe/src/lib.rs

@@ -87,7 +87,6 @@ use futures::{
 use log::{debug, info, log, trace, warn};
 use parking_lot::Mutex;
 use prometheus_endpoint::Registry;
-use retain_mut::RetainMut;
 use schnorrkel::SignatureError;
 
 use sc_client_api::{
@@ -835,17 +834,16 @@ where
 		slot: Slot,
 		epoch_descriptor: &ViableEpochDescriptor<B::Hash, NumberFor<B>, Epoch>,
 	) {
-		RetainMut::retain_mut(&mut *self.slot_notification_sinks.lock(), |sink| {
+		let sinks = &mut self.slot_notification_sinks.lock();
-			match sink.try_send((slot, epoch_descriptor.clone())) {
+		sinks.retain_mut(|sink| match sink.try_send((slot, epoch_descriptor.clone())) {
-				Ok(()) => true,
+			Ok(()) => true,
-				Err(e) =>
+			Err(e) =>
-					if e.is_full() {
+				if e.is_full() {
-						warn!(target: "babe", "Trying to notify a slot but the channel is full");
+					warn!(target: "babe", "Trying to notify a slot but the channel is full");
-						true
+					true
-					} else {
+				} else {
-						false
+					false
-					},
+				},
-			}
 		});
 	}
 

substrate/client/transaction-pool/Cargo.toml

@@ -20,7 +20,6 @@ linked-hash-map = "0.5.4"
 log = "0.4.17"
 parity-util-mem = { version = "0.11.0", default-features = false, features = ["primitive-types"] }
 parking_lot = "0.12.0"
-retain_mut = "0.1.4"
 serde = { version = "1.0.136", features = ["derive"] }
 thiserror = "1.0.30"
 prometheus-endpoint = { package = "substrate-prometheus-endpoint", version = "0.10.0-dev", path = "../../utils/prometheus" }

substrate/client/transaction-pool/src/graph/validated_pool.rs

@@ -24,7 +24,6 @@ use std::{
 
 use futures::channel::mpsc::{channel, Sender};
 use parking_lot::{Mutex, RwLock};
-use retain_mut::RetainMut;
 use sc_transaction_pool_api::{error, PoolStatus, ReadyTransactions};
 use serde::Serialize;
 use sp_runtime::{
@@ -204,21 +203,20 @@ impl<B: ChainApi> ValidatedPool<B> {
 				let imported = self.pool.write().import(tx)?;
 
 				if let base::Imported::Ready { ref hash, .. } = imported {
-					RetainMut::retain_mut(&mut *self.import_notification_sinks.lock(), |sink| {
+					let sinks = &mut self.import_notification_sinks.lock();
-						match sink.try_send(*hash) {
+					sinks.retain_mut(|sink| match sink.try_send(*hash) {
-							Ok(()) => true,
+						Ok(()) => true,
-							Err(e) =>
+						Err(e) =>
-								if e.is_full() {
+							if e.is_full() {
-									log::warn!(
+								log::warn!(
-										target: "txpool",
+									target: "txpool",
-										"[{:?}] Trying to notify an import but the channel is full",
+									"[{:?}] Trying to notify an import but the channel is full",
-										hash,
+									hash,
-									);
+								);
-									true
+								true
-								} else {
+							} else {
-									false
+								false
-								},
+							},
-						}
 					});
 				}