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 69d5e72e563d7cf29d745c5be37d9c6d5eb42119 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Wed May 22 08:47:37 2024 -0400 Fix PMD UnnecessaryFullyQualifiedName in ReferenceMapIterator --- src/changes/changes.xml | 1 + .../org/apache/commons/collections4/map/AbstractReferenceMap.java | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/changes/changes.xml b/src/changes/changes.xml index 2e89874b5..c67b46de6 100644 --- a/src/changes/changes.xml +++ b/src/changes/changes.xml @@ -42,6 +42,7 @@ <action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in IndexUtils.</action> <action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in AbstractLinkedMap.</action> <action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in LinkIterator.</action> + <action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in ReferenceMapIterator.</action> <!-- UPDATE --> <action type="update" dev="ggregory" due-to="Dependabot">Bump org.apache.commons:commons-parent from 67 to 69 #473.</action> <action type="update" dev="ggregory" due-to="Dependabot">Bump tests commons-io:commons-io from 2.16.0 to 2.16.1 #475.</action> diff --git a/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java b/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java index 7202eadf9..e08302356 100644 --- a/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java +++ b/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java @@ -465,7 +465,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V> public K getKey() { final HashEntry<K, V> current = currentEntry(); if (current == null) { - throw new IllegalStateException(AbstractHashedMap.GETKEY_INVALID); + throw new IllegalStateException(GETKEY_INVALID); } return current.getKey(); } @@ -474,7 +474,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V> public V getValue() { final HashEntry<K, V> current = currentEntry(); if (current == null) { - throw new IllegalStateException(AbstractHashedMap.GETVALUE_INVALID); + throw new IllegalStateException(GETVALUE_INVALID); } return current.getValue(); } @@ -488,7 +488,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V> public V setValue(final V value) { final HashEntry<K, V> current = currentEntry(); if (current == null) { - throw new IllegalStateException(AbstractHashedMap.SETVALUE_INVALID); + throw new IllegalStateException(SETVALUE_INVALID); } return current.setValue(value); }