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); } }