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-collections.git
The following commit(s) were added to refs/heads/master by this push:
new e78f814ec Flip null test
e78f814ec is described below
commit e78f814ec86f3a85cfa1b0f5e07e4fbf2ed24a97
Author: Gary Gregory <[email protected]>
AuthorDate: Sun Jul 5 09:46:26 2026 -0400
Flip null test
---
src/main/java/org/apache/commons/collections4/CollectionUtils.java | 2 +-
.../org/apache/commons/collections4/comparators/ComparatorChain.java | 2 +-
.../org/apache/commons/collections4/comparators/ReverseComparator.java | 2 +-
.../apache/commons/collections4/comparators/TransformingComparator.java | 2 +-
src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/main/java/org/apache/commons/collections4/CollectionUtils.java
b/src/main/java/org/apache/commons/collections4/CollectionUtils.java
index b3019eff3..7813dd49e 100644
--- a/src/main/java/org/apache/commons/collections4/CollectionUtils.java
+++ b/src/main/java/org/apache/commons/collections4/CollectionUtils.java
@@ -1141,7 +1141,7 @@ public class CollectionUtils {
public static <E> int hashCode(final Collection<? extends E> collection,
final Equator<? super E> equator) {
Objects.requireNonNull(equator, "equator");
- if (null == collection) {
+ if (collection == null) {
return 0;
}
int hashCode = 1;
diff --git
a/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java
b/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java
index 2ef66bae2..6ef4cb21d 100644
---
a/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java
+++
b/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java
@@ -242,7 +242,7 @@ public class ComparatorChain<E> implements Comparator<E>,
Serializable {
if (this == object) {
return true;
}
- if (null == object) {
+ if (object == null) {
return false;
}
if (object.getClass().equals(this.getClass())) {
diff --git
a/src/main/java/org/apache/commons/collections4/comparators/ReverseComparator.java
b/src/main/java/org/apache/commons/collections4/comparators/ReverseComparator.java
index c02172a32..d9d50a194 100644
---
a/src/main/java/org/apache/commons/collections4/comparators/ReverseComparator.java
+++
b/src/main/java/org/apache/commons/collections4/comparators/ReverseComparator.java
@@ -94,7 +94,7 @@ public class ReverseComparator<E> implements Comparator<E>,
Serializable {
if (this == object) {
return true;
}
- if (null == object) {
+ if (object == null) {
return false;
}
if (object.getClass().equals(this.getClass())) {
diff --git
a/src/main/java/org/apache/commons/collections4/comparators/TransformingComparator.java
b/src/main/java/org/apache/commons/collections4/comparators/TransformingComparator.java
index 91cd93837..feae236ce 100644
---
a/src/main/java/org/apache/commons/collections4/comparators/TransformingComparator.java
+++
b/src/main/java/org/apache/commons/collections4/comparators/TransformingComparator.java
@@ -101,7 +101,7 @@ public class TransformingComparator<I, O> implements
Comparator<I>, Serializable
if (this == object) {
return true;
}
- if (null == object) {
+ if (object == null) {
return false;
}
if (object.getClass().equals(this.getClass())) {
diff --git
a/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java
b/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java
index 5e318daef..52de89e95 100644
--- a/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java
+++ b/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java
@@ -688,7 +688,7 @@ public final class StaticBucketMap<K, V> extends
AbstractIterableMap<K, V> {
while (n != null) {
if (Objects.equals(n.key, key)) {
// Remove this node from the linked list of nodes.
- if (null == prev) {
+ if (prev == null) {
// This node was the head, set the next node to be the
new head.
buckets[hash] = n.next;
} else {
