From: Elias Rohrer <ero@tnull.de>
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`",