This is an automated email from the ASF dual-hosted git repository.
garydgregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-net.git
The following commit(s) were added to refs/heads/master by this push:
new a3d19751 Flip null test
a3d19751 is described below
commit a3d19751891edb9f7cb8c11bc344eaf1683b2b32
Author: Gary Gregory <[email protected]>
AuthorDate: Sun Jul 5 10:00:15 2026 -0400
Flip null test
---
.../commons/net/ftp/parser/ConfigurableFTPFileEntryParserImpl.java | 4 ++--
.../org/apache/commons/net/ftp/parser/MacOsPeterFTPEntryParser.java | 2 +-
src/main/java/org/apache/commons/net/ftp/parser/NTFTPEntryParser.java | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git
a/src/main/java/org/apache/commons/net/ftp/parser/ConfigurableFTPFileEntryParserImpl.java
b/src/main/java/org/apache/commons/net/ftp/parser/ConfigurableFTPFileEntryParserImpl.java
index d78df467..866883cc 100644
---
a/src/main/java/org/apache/commons/net/ftp/parser/ConfigurableFTPFileEntryParserImpl.java
+++
b/src/main/java/org/apache/commons/net/ftp/parser/ConfigurableFTPFileEntryParserImpl.java
@@ -72,10 +72,10 @@ public abstract class ConfigurableFTPFileEntryParserImpl
extends RegexFTPFileEnt
if (timestampParser instanceof Configurable) {
final FTPClientConfig defaultCfg = getDefaultConfiguration();
if (config != null) {
- if (null == config.getDefaultDateFormatStr()) {
+ if (config.getDefaultDateFormatStr() == null) {
config.setDefaultDateFormatStr(defaultCfg.getDefaultDateFormatStr());
}
- if (null == config.getRecentDateFormatStr()) {
+ if (config.getRecentDateFormatStr() == null) {
config.setRecentDateFormatStr(defaultCfg.getRecentDateFormatStr());
}
((Configurable) timestampParser).configure(config);
diff --git
a/src/main/java/org/apache/commons/net/ftp/parser/MacOsPeterFTPEntryParser.java
b/src/main/java/org/apache/commons/net/ftp/parser/MacOsPeterFTPEntryParser.java
index 60427d68..556dc0d8 100644
---
a/src/main/java/org/apache/commons/net/ftp/parser/MacOsPeterFTPEntryParser.java
+++
b/src/main/java/org/apache/commons/net/ftp/parser/MacOsPeterFTPEntryParser.java
@@ -176,7 +176,7 @@ public class MacOsPeterFTPEntryParser extends
ConfigurableFTPFileEntryParserImpl
// intentionally do nothing
}
- if (null == endtoken) {
+ if (endtoken == null) {
file.setName(name);
} else {
// oddball cases like symbolic links, file names
diff --git
a/src/main/java/org/apache/commons/net/ftp/parser/NTFTPEntryParser.java
b/src/main/java/org/apache/commons/net/ftp/parser/NTFTPEntryParser.java
index 95233888..08de000e 100644
--- a/src/main/java/org/apache/commons/net/ftp/parser/NTFTPEntryParser.java
+++ b/src/main/java/org/apache/commons/net/ftp/parser/NTFTPEntryParser.java
@@ -99,7 +99,7 @@ public class NTFTPEntryParser extends
ConfigurableFTPFileEntryParserImpl {
final String dirString = group(3);
final String size = group(4);
final String name = group(5);
- if (null == name || name.equals(".") || name.equals("..")) {
+ if (name == null || name.equals(".") || name.equals("..")) {
return null;
}
try {
