(commons-jexl) branch master updated: Add messages when throwing NullPointerException.

[ggregory]

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-jexl.git


The following commit(s) were added to refs/heads/master by this push:
     new 765d80bf Add messages when throwing NullPointerException.
765d80bf is described below

commit 765d80bf532949787505fec68d1d0aa3657750a5
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Jul 22 15:13:55 2026 -0400

    Add messages when throwing NullPointerException.
---
 src/changes/changes.xml                                               | 3 ++-
 src/main/java/org/apache/commons/jexl3/JexlException.java             | 4 ++--
 .../java/org/apache/commons/jexl3/introspection/JexlPermissions.java  | 4 ++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 665ad449..f79a5b79 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -30,7 +30,8 @@
         <release version="3.7.1" date="YYYY-MM-DD" description="This is a 
feature and maintenance release. Java 8 or later is required.">
             <!-- ADD -->
             <!-- FIX -->
-            <action dev="ggregory" type="fix">Pick up commons.jacoco.version 
from the parent POM.</action>
+            <action dev="ggregory" type="fix" due-to="Gary Gregory">Pick up 
commons.jacoco.version from the parent POM.</action>
+            <action dev="ggregory" type="fix" due-to="Gary Gregory">Add 
messages when throwing NullPointerException.</action>
             <!-- UPDATE -->
             <action type="update" dev="ggregory" due-to="Gary Gregory">Bump 
org.apache.commons:commons-parent from 102 to 103.</action>
         </release>
diff --git a/src/main/java/org/apache/commons/jexl3/JexlException.java 
b/src/main/java/org/apache/commons/jexl3/JexlException.java
index 122c4357..e51a46a9 100644
--- a/src/main/java/org/apache/commons/jexl3/JexlException.java
+++ b/src/main/java/org/apache/commons/jexl3/JexlException.java
@@ -381,7 +381,7 @@ public class JexlException extends RuntimeException {
          * @param cause The javacc cause
          */
         public Parsing(final JexlInfo info, final ParseException cause) {
-            super(merge(info, cause), 
Objects.requireNonNull(cause).getAfter(), null);
+            super(merge(info, cause), Objects.requireNonNull(cause, 
"cause").getAfter(), null);
         }
 
         /**
@@ -580,7 +580,7 @@ public class JexlException extends RuntimeException {
          * @param cause The javacc cause
          */
         public Tokenization(final JexlInfo info, final TokenMgrException 
cause) {
-            super(merge(info, cause), 
Objects.requireNonNull(cause).getAfter(), null);
+            super(merge(info, cause), Objects.requireNonNull(cause, 
"cause").getAfter(), null);
         }
 
         @Override
diff --git 
a/src/main/java/org/apache/commons/jexl3/introspection/JexlPermissions.java 
b/src/main/java/org/apache/commons/jexl3/introspection/JexlPermissions.java
index b1f8d6f7..dafa4a03 100644
--- a/src/main/java/org/apache/commons/jexl3/introspection/JexlPermissions.java
+++ b/src/main/java/org/apache/commons/jexl3/introspection/JexlPermissions.java
@@ -128,7 +128,7 @@ public interface JexlPermissions {
        * @param allow The set of allowed classes
        */
       public ClassPermissions(final JexlPermissions permissions, final 
Class<?>... allow) {
-        this(permissions, 
Arrays.stream(Objects.requireNonNull(allow)).map(Class::getCanonicalName).collect(Collectors.toList()));
+          this(permissions, Arrays.stream(Objects.requireNonNull(allow, 
"allow")).map(Class::getCanonicalName).collect(Collectors.toList()));
       }
 
       /**
@@ -138,7 +138,7 @@ public interface JexlPermissions {
        * @param allow    The list of class canonical names
        */
       public ClassPermissions(final JexlPermissions delegate, final 
Collection<String> allow) {
-        super(Objects.requireNonNull(delegate));
+        super(Objects.requireNonNull(delegate, "delegate"));
         allowedClasses = new HashSet<>(Objects.requireNonNull(allow));
       }