You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2021/01/28 15:42:21 UTC
[commons-net] 02/02: Simplify by inverting condition
This is an automated email from the ASF dual-hosted git repository.
sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-net.git
commit 9d17fd7f7b9821a24638d0de2256a8b62092e0e0
Author: Sebb <se...@apache.org>
AuthorDate: Thu Jan 28 15:42:09 2021 +0000
Simplify by inverting condition
---
src/main/java/org/apache/commons/net/ftp/FTPClient.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/main/java/org/apache/commons/net/ftp/FTPClient.java b/src/main/java/org/apache/commons/net/ftp/FTPClient.java
index f01c928..7490693 100644
--- a/src/main/java/org/apache/commons/net/ftp/FTPClient.java
+++ b/src/main/java/org/apache/commons/net/ftp/FTPClient.java
@@ -619,8 +619,7 @@ implements Configurable
delim3 = reply.charAt(2);
delim4 = reply.charAt(reply.length()-1);
- if (!(delim1 == delim2) || !(delim2 == delim3)
- || !(delim3 == delim4)) {
+ if ((delim1 != delim2) || (delim2 != delim3) || (delim3 != delim4)) {
throw new MalformedServerReplyException(
"Could not parse extended passive host information.\nServer Reply: " + reply);
}