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-configuration.git
The following commit(s) were added to refs/heads/master by this push:
new aad1d6ba0 Add messages when throwing NullPointerException.
aad1d6ba0 is described below
commit aad1d6ba083f6a9155174ba7969031167bf974e9
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Jul 22 11:07:53 2026 -0400
Add messages when throwing NullPointerException.
---
src/changes/changes.xml | 1 +
.../org/apache/commons/configuration2/ImmutableConfiguration.java | 2 +-
.../java/org/apache/commons/configuration2/MapConfiguration.java | 2 +-
.../commons/configuration2/io/TestCombinedLocationStrategy.java | 6 +++---
4 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 3fc2ec1b6..453e51cf4 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -25,6 +25,7 @@
<body>
<release version="2.15.2" date="YYYY-MM-DD" description="This is a feature
and maintenance release. Java 8 or later is required.">
<!-- FIX -->
+ <action type="fix" dev="ggregory" due-to="Gary Gregory">Add messages
when throwing NullPointerException.</action>
<!-- ADD -->
<!-- UPDATE -->
<action type="update" dev="ggregory" due-to="Gary Gregory">Bump
org.apache.commons:commons-parent from 100 to 103.</action>
diff --git
a/src/main/java/org/apache/commons/configuration2/ImmutableConfiguration.java
b/src/main/java/org/apache/commons/configuration2/ImmutableConfiguration.java
index 3d3372b95..d7715fee8 100644
---
a/src/main/java/org/apache/commons/configuration2/ImmutableConfiguration.java
+++
b/src/main/java/org/apache/commons/configuration2/ImmutableConfiguration.java
@@ -119,7 +119,7 @@ public interface ImmutableConfiguration {
* @since 2.13.0
*/
default void forEach(final BiConsumer<String, Object> action) {
- Objects.requireNonNull(action);
+ Objects.requireNonNull(action, "action");
for (final Map.Entry<String, Object> entry : entrySet()) {
String k;
Object v;
diff --git
a/src/main/java/org/apache/commons/configuration2/MapConfiguration.java
b/src/main/java/org/apache/commons/configuration2/MapConfiguration.java
index 5c6a1d446..17768a910 100644
--- a/src/main/java/org/apache/commons/configuration2/MapConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/MapConfiguration.java
@@ -117,7 +117,7 @@ public class MapConfiguration extends AbstractConfiguration
implements Cloneable
* @since 1.8
*/
public MapConfiguration(final Properties props) {
- map = toMap(Objects.requireNonNull(props));
+ map = toMap(Objects.requireNonNull(props, "props"));
}
@Override
diff --git
a/src/test/java/org/apache/commons/configuration2/io/TestCombinedLocationStrategy.java
b/src/test/java/org/apache/commons/configuration2/io/TestCombinedLocationStrategy.java
index 0ebaa08f6..18fc55242 100644
---
a/src/test/java/org/apache/commons/configuration2/io/TestCombinedLocationStrategy.java
+++
b/src/test/java/org/apache/commons/configuration2/io/TestCombinedLocationStrategy.java
@@ -133,9 +133,9 @@ public class TestCombinedLocationStrategy {
// Create a collection that throws NPE on contains(null) like
List.of() instance does
final Collection<FileLocationStrategy> collectionThatThrowsNPE = new
ArrayList<FileLocationStrategy>(Arrays.asList(getSubStrategies())) {
@Override
- public boolean contains(final Object o) {
- Objects.requireNonNull(o);
- return super.contains(o);
+ public boolean contains(final Object obj) {
+ Objects.requireNonNull(obj, "obj");
+ return super.contains(obj);
}
};
