You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/06/16 20:06:56 UTC

[commons-configuration] 02/03: Throwing Exception is an anti-pattern, so be more precise since we easily can

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

commit 84205a8df7f76119b46559220102a83cb46ba42c
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Jun 16 16:03:17 2022 -0400

    Throwing Exception is an anti-pattern, so be more precise since we
    easily can
---
 .../configuration2/builder/BuilderConfigurationWrapperFactory.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java b/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java
index 1f8cd9ff..dff5ed2b 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java
@@ -23,6 +23,7 @@ import java.lang.reflect.Proxy;
 import org.apache.commons.configuration2.ConfigurationUtils;
 import org.apache.commons.configuration2.ImmutableConfiguration;
 import org.apache.commons.configuration2.event.EventSource;
+import org.apache.commons.configuration2.ex.ConfigurationException;
 
 /**
  * <p>
@@ -226,7 +227,7 @@ public class BuilderConfigurationWrapperFactory {
          * @return the return value of the method
          * @throws Exception if an error occurs
          */
-        private Object handleConfigurationInvocation(final Method method, final Object[] args) throws Exception {
+        private Object handleConfigurationInvocation(final Method method, final Object[] args) throws ReflectiveOperationException, ConfigurationException {
             return method.invoke(builder.getConfiguration(), args);
         }