From c90a1f70a6ec441790829d346fcadf24906155c5 Mon Sep 17 00:00:00 2001 From: Steve Myers Date: Thu, 11 Feb 2021 13:08:21 -0800 Subject: [PATCH] Fix clippy warn on compact_filters peer::_recv() --- src/blockchain/compact_filters/peer.rs | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/src/blockchain/compact_filters/peer.rs b/src/blockchain/compact_filters/peer.rs index 62fa2cb4..754c832b 100644 --- a/src/blockchain/compact_filters/peer.rs +++ b/src/blockchain/compact_filters/peer.rs @@ -223,14 +223,14 @@ impl Peer { )), )?; let version = if let NetworkMessage::Version(version) = - Self::_recv(&responses, "version", None)?.unwrap() + Self::_recv(&responses, "version", None).unwrap() { version } else { return Err(CompactFiltersError::InvalidResponse); }; - if let NetworkMessage::Verack = Self::_recv(&responses, "verack", None)?.unwrap() { + if let NetworkMessage::Verack = Self::_recv(&responses, "verack", None).unwrap() { Self::_send(&mut locked_writer, network.magic(), NetworkMessage::Verack)?; } else { return Err(CompactFiltersError::InvalidResponse); @@ -271,7 +271,7 @@ impl Peer { responses: &Arc>, wait_for: &'static str, timeout: Option, - ) -> Result, CompactFiltersError> { + ) -> Option { let message_resp = { let mut lock = responses.write().unwrap(); let message_resp = lock.entry(wait_for).or_default(); @@ -287,15 +287,14 @@ impl Peer { Some(t) => { let result = cvar.wait_timeout(messages, t).unwrap(); if result.1.timed_out() { - return Ok(None); + return None; } - messages = result.0; } } } - Ok(messages.pop()) + messages.pop() } /// Return the [`VersionMessage`] sent by the peer @@ -415,7 +414,7 @@ impl Peer { wait_for: &'static str, timeout: Option, ) -> Result, CompactFiltersError> { - Self::_recv(&self.responses, wait_for, timeout) + Ok(Self::_recv(&self.responses, wait_for, timeout)) } }