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 <s...@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);
         }

Reply via email to