This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-collections.git
commit d51fff2bddfecca83c5231e50ec30772ddea82c3 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sun Jun 23 14:17:17 2024 -0400 Use if-else --- .../java/org/apache/commons/collections4/map/AbstractHashedMap.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/apache/commons/collections4/map/AbstractHashedMap.java b/src/main/java/org/apache/commons/collections4/map/AbstractHashedMap.java index 1dc9258ca..8e458ad97 100644 --- a/src/main/java/org/apache/commons/collections4/map/AbstractHashedMap.java +++ b/src/main/java/org/apache/commons/collections4/map/AbstractHashedMap.java @@ -1030,10 +1030,8 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab if (map.get(key) != null || !map.containsKey(key)) { return false; } - } else { - if (!value.equals(map.get(key))) { - return false; - } + } else if (!value.equals(map.get(key))) { + return false; } } } catch (final ClassCastException | NullPointerException ignored) {