From: Elias Rohrer Date: Mon, 5 Sep 2022 10:18:30 +0000 (+0200) Subject: Remove redundant `is_none()` check X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=64d7bdce16ae5a6d83cc290551b42574db177193;p=ldk-sample Remove redundant `is_none()` check We don't need to check the same iterator twice. And, as at this point pubkey will always be `Some(..)`, we don't need to check it either. We therefore can just remove the first part of the condition. --- diff --git a/src/cli.rs b/src/cli.rs index 013209f..5863ed0 100644 --- a/src/cli.rs +++ b/src/cli.rs @@ -762,7 +762,7 @@ pub(crate) fn parse_peer_info( let mut pubkey_and_addr = peer_pubkey_and_ip_addr.split("@"); let pubkey = pubkey_and_addr.next(); let peer_addr_str = pubkey_and_addr.next(); - if peer_addr_str.is_none() || peer_addr_str.is_none() { + if peer_addr_str.is_none() { return Err(std::io::Error::new( std::io::ErrorKind::Other, "ERROR: incorrectly formatted peer info. Should be formatted as: `pubkey@host:port`",