[commons-digester] branch master updated: Throw a RuntimeException subclass instead of RuntimeException

[ggregory]

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


The following commit(s) were added to refs/heads/master by this push:
     new f4eb40c5 Throw a RuntimeException subclass instead of RuntimeException
f4eb40c5 is described below

commit f4eb40c5fe9ff22c074fe9d0a8bd481b7f96d90f
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Thu Jul 6 15:22:39 2023 -0400

    Throw a RuntimeException subclass instead of RuntimeException
---
 core/src/main/java/org/apache/commons/digester3/CallMethodRule.java     | 2 +-
 .../org/apache/commons/digester3/binder/NodeCreateRuleProvider.java     | 2 +-
 .../main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java  | 2 +-
 .../main/java/org/apache/commons/digester3/xmlrules/NodeCreateRule.java | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/core/src/main/java/org/apache/commons/digester3/CallMethodRule.java 
b/core/src/main/java/org/apache/commons/digester3/CallMethodRule.java
index 4534d409..e0c4e13f 100644
--- a/core/src/main/java/org/apache/commons/digester3/CallMethodRule.java
+++ b/core/src/main/java/org/apache/commons/digester3/CallMethodRule.java
@@ -328,7 +328,7 @@ public class CallMethodRule
                 }
                 catch ( final ClassNotFoundException e )
                 {
-                    throw new RuntimeException( format( "[CallMethodRule] 
Cannot load class %s at position %s",
+                    throw new IllegalArgumentException( format( 
"[CallMethodRule] Cannot load class %s at position %s",
                                                         
this.paramClassNames[i], i ), e );
                 }
             }
diff --git 
a/core/src/main/java/org/apache/commons/digester3/binder/NodeCreateRuleProvider.java
 
b/core/src/main/java/org/apache/commons/digester3/binder/NodeCreateRuleProvider.java
index 32c6a7da..ae7a1fa1 100644
--- 
a/core/src/main/java/org/apache/commons/digester3/binder/NodeCreateRuleProvider.java
+++ 
b/core/src/main/java/org/apache/commons/digester3/binder/NodeCreateRuleProvider.java
@@ -104,7 +104,7 @@ public final class NodeCreateRuleProvider
             }
             catch ( final ParserConfigurationException e )
             {
-                throw new RuntimeException( e );
+                throw new IllegalStateException( e );
             }
         }
 
diff --git 
a/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java 
b/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java
index c03e0389..e0adfd7f 100644
--- 
a/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java
+++ 
b/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java
@@ -65,7 +65,7 @@ final class CallParamRule
         {
             if ( fromStack != null ) {
                 // specifying both from-stack and attribute name is not allowed
-                throw new RuntimeException( "Attributes from-stack and 
attrname cannot both be present." );
+                throw new IllegalArgumentException( "Attributes from-stack and 
attrname cannot both be present." );
             }
             builder.fromAttribute( attributeName );
         }
diff --git 
a/core/src/main/java/org/apache/commons/digester3/xmlrules/NodeCreateRule.java 
b/core/src/main/java/org/apache/commons/digester3/xmlrules/NodeCreateRule.java
index bc7fa62d..e8073483 100644
--- 
a/core/src/main/java/org/apache/commons/digester3/xmlrules/NodeCreateRule.java
+++ 
b/core/src/main/java/org/apache/commons/digester3/xmlrules/NodeCreateRule.java
@@ -61,7 +61,7 @@ final class NodeCreateRule
             }
             else
             {
-                throw new RuntimeException( format(
+                throw new IllegalArgumentException( format(
                     "Unrecognized node type: %s. This attribute is optional or 
can have a value of element|fragment.",
                     nodeType ) );
             }