Nice, thanks!

Sent from my iPhone

> On 2016/02/16, at 4:47, mattsic...@apache.org wrote:
> 
> Use Class.asSubclass() to remove @SuppressWarnings.
> 
> 
> Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
> Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/e1134a14
> Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/e1134a14
> Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/e1134a14
> 
> Branch: refs/heads/master
> Commit: e1134a148231681da566963fdfac35a528b88cd6
> Parents: 009ad22
> Author: Matt Sicker <boa...@gmail.com>
> Authored: Mon Feb 15 13:46:57 2016 -0600
> Committer: Matt Sicker <boa...@gmail.com>
> Committed: Mon Feb 15 13:46:57 2016 -0600
> 
> ----------------------------------------------------------------------
> .../logging/log4j/core/config/ReliabilityStrategyFactory.java    | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> ----------------------------------------------------------------------
> 
> 
> http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/e1134a14/log4j-core/src/main/java/org/apache/logging/log4j/core/config/ReliabilityStrategyFactory.java
> ----------------------------------------------------------------------
> diff --git 
> a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/ReliabilityStrategyFactory.java
>  
> b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/ReliabilityStrategyFactory.java
> index 773f9f0..637341b 100644
> --- 
> a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/ReliabilityStrategyFactory.java
> +++ 
> b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/ReliabilityStrategyFactory.java
> @@ -58,8 +58,8 @@ public final class ReliabilityStrategyFactory {
>             return new LockingReliabilityStrategy(loggerConfig);
>         }
>         try {
> -            @SuppressWarnings("unchecked")
> -            Class<? extends ReliabilityStrategy> cls = (Class<? extends 
> ReliabilityStrategy>) LoaderUtil.loadClass(strategy);
> +            final Class<? extends ReliabilityStrategy> cls = 
> LoaderUtil.loadClass(strategy).asSubclass(
> +                ReliabilityStrategy.class);
>             return 
> cls.getConstructor(LoggerConfig.class).newInstance(loggerConfig);
>         } catch (Exception dynamicFailed) {
>             StatusLogger.getLogger().warn(
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscr...@logging.apache.org
For additional commands, e-mail: log4j-dev-h...@logging.apache.org

Reply via email to