Apache
/
commons-collections
mirror of https://github.com/apache/commons-collections.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix PMD UnnecessaryFullyQualifiedName in ReferenceMapIterator

This commit is contained in:
Gary Gregory 2024-05-22 08:47:37 -04:00
parent 520fe0c7a0
commit 69d5e72e56
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/changes/changes.xml
View File

@ -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 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 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 LinkIterator.</action>
<action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in ReferenceMapIterator.</action>
<!-- UPDATE --> <!-- 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 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> <action type="update" dev="ggregory" due-to="Dependabot">Bump tests commons-io:commons-io from 2.16.0 to 2.16.1 #475.</action>

6
src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java
View File

@ -465,7 +465,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V>
public K getKey() { public K getKey() {
final HashEntry<K, V> current = currentEntry(); final HashEntry<K, V> current = currentEntry();
if (current == null) { if (current == null) {
throw new IllegalStateException(AbstractHashedMap.GETKEY_INVALID); throw new IllegalStateException(GETKEY_INVALID);
} }
return current.getKey(); return current.getKey();
} }
@ -474,7 +474,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V>
public V getValue() { public V getValue() {
final HashEntry<K, V> current = currentEntry(); final HashEntry<K, V> current = currentEntry();
if (current == null) { if (current == null) {
throw new IllegalStateException(AbstractHashedMap.GETVALUE_INVALID); throw new IllegalStateException(GETVALUE_INVALID);
} }
return current.getValue(); return current.getValue();
} }
@ -488,7 +488,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V>
public V setValue(final V value) { public V setValue(final V value) {
final HashEntry<K, V> current = currentEntry(); final HashEntry<K, V> current = currentEntry();
if (current == null) { if (current == null) {
throw new IllegalStateException(AbstractHashedMap.SETVALUE_INVALID); throw new IllegalStateException(SETVALUE_INVALID);
} }
return current.setValue(value); return current.setValue(value);
} }