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 AbstractHashedMap

This commit is contained in:
Gary Gregory 2024-05-22 08:38:21 -04:00
parent 7880a99f5e
commit 8c1939abcc
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/changes/changes.xml
View File

@ -37,6 +37,7 @@
<action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in PredicateUtils.</action>
<action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in TransformerUtils.</action>
<action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in DefaultEquator.</action>
<action type="update" dev="ggregory" due-to="PMD, Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in AbstractHashedMap.</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>

10
src/main/java/org/apache/commons/collections4/map/AbstractHashedMap.java
View File

@ -251,7 +251,7 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab
}
final HashEntry<K, V> newCurrent = next;
if (newCurrent == null) {
throw new NoSuchElementException(AbstractHashedMap.NO_NEXT_ENTRY);
throw new NoSuchElementException(NO_NEXT_ENTRY);
}
final HashEntry<K, V>[] data = parent.data;
int i = hashIndex;
@ -267,7 +267,7 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab
public void remove() {
if (last == null) {
throw new IllegalStateException(AbstractHashedMap.REMOVE_INVALID);
throw new IllegalStateException(REMOVE_INVALID);
}
if (parent.modCount != expectedModCount) {
throw new ConcurrentModificationException();
@ -301,7 +301,7 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab
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();
}
@ -310,7 +310,7 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab
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();
}
@ -324,7 +324,7 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab
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);
}