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