You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by Remko Popma <re...@gmail.com> on 2016/02/15 23:52:39 UTC
Re: [2/2] logging-log4j2 git commit: Use Class.asSubclass() to remove @SuppressWarnings.
Nice, thanks!
Sent from my iPhone
> On 2016/02/16, at 4:47, mattsicker@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 <bo...@gmail.com>
> Authored: Mon Feb 15 13:46:57 2016 -0600
> Committer: Matt Sicker <bo...@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-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org