fixed some comments

bin/node/runtime/src/lib.rs

@@ -1613,17 +1613,10 @@ impl_runtime_apis! {
 		fn verify_proof(leaves: Vec<mmr::EncodableOpaqueLeaf>, proof: mmr::Proof<mmr::Hash>)
 			-> Result<(), mmr::Error>
 		{
-			let leaves = {
-				let mut decoded_leaves = Vec::new();
-				for l in &leaves {
-					let leaf: mmr::Leaf  = l.clone()
-						.into_opaque_leaf()
-						.try_decode()
-						.ok_or(mmr::Error::Verify)?;
-					decoded_leaves.push(leaf);
-				}
-				decoded_leaves
-			};
+			let leaves = leaves.into_iter().map(|leaf|
+				leaf.into_opaque_leaf()
+				.try_decode()
+				.ok_or(mmr::Error::Verify)).collect::<Result<Vec<mmr::Leaf>, mmr::Error>>()?;
 
 			Mmr::verify_leaves(leaves, proof)
 		}

frame/merkle-mountain-range/src/mmr/mmr.rs

@@ -50,10 +50,10 @@ where
 	);
 
 	let mut verification_result = Ok(false);
-	let is_valid = leaves.iter().all(|leaf| {
+	let is_valid = leaves.into_iter().all(|leaf| {
 		let mut res = false;
 		for pos in &positions {
-			match p.verify(Node::Hash(root.clone()), vec![(*pos, leaf.clone())]) {
+			match p.verify(Node::Hash(root.clone()), vec![(*pos, leaf)]) {
 				Ok(val) => {
 					res = val;
 					break
@@ -122,10 +122,10 @@ where
 		let root = self.mmr.get_root().map_err(|e| Error::GetRoot.log_error(e))?;
 		let mut verification_result = Ok(false);
 
-		let is_valid = leaves.iter().all(|leaf| {
+		let is_valid = leaves.into_iter().all(|leaf| {
 			let mut res = false;
 			for pos in &positions {
-				match p.verify(root.clone(), vec![(*pos, Node::Data(leaf.clone()))]) {
+				match p.verify(root.clone(), vec![(*pos, Node::Data(leaf))]) {
 					Ok(val) => {
 						res = val;
 						break