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


The following commit(s) were added to refs/heads/master by this push:
     new 7293cf98 
org.apache.commons.jexl3.internal.introspection.AbstractExecutor.initMarker(Class,
 String, Class...) throws IllegalArgumentException instead of Error
7293cf98 is described below

commit 7293cf985a7af74bf732c56d493dee6826bc898c
Author: Gary D. Gregory <garydgreg...@gmail.com>
AuthorDate: Fri May 2 20:50:32 2025 -0400

    
org.apache.commons.jexl3.internal.introspection.AbstractExecutor.initMarker(Class,
    String, Class...) throws IllegalArgumentException instead of Error
---
 src/changes/changes.xml                                               | 1 +
 .../apache/commons/jexl3/internal/introspection/AbstractExecutor.java | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index b0770575..c77497f5 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -29,6 +29,7 @@
     <body>
         <release version="3.5.1" 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">org.apache.commons.jexl3.internal.introspection.AbstractExecutor.initMarker(Class,
 String, Class...) throws IllegalArgumentException instead of Error.</action>
           <!-- ADD -->
           <!-- UPDATE -->
           <action type="update" dev="ggregory" due-to="Gary Gregory, 
Dependabot">Bump org.apache.commons:commons-parent from 81 to 83 #344.</action>
diff --git 
a/src/main/java/org/apache/commons/jexl3/internal/introspection/AbstractExecutor.java
 
b/src/main/java/org/apache/commons/jexl3/internal/introspection/AbstractExecutor.java
index eb503460..151149a3 100644
--- 
a/src/main/java/org/apache/commons/jexl3/internal/introspection/AbstractExecutor.java
+++ 
b/src/main/java/org/apache/commons/jexl3/internal/introspection/AbstractExecutor.java
@@ -125,8 +125,8 @@ abstract class AbstractExecutor {
     static java.lang.reflect.Method initMarker(final Class<?> clazz, final 
String name, final Class<?>... parms) {
         try {
             return clazz.getMethod(name, parms);
-        } catch (final Exception xnever) {
-            throw new Error(xnever);
+        } catch (final Exception e) {
+            throw new IllegalArgumentException(e);
         }
     }
 

Reply via email to