Repository: logging-log4j2 Updated Branches: refs/heads/master b4ae3fdc7 -> b50673ed8
Don't use unnecessary else clause. Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/b50673ed Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/b50673ed Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/b50673ed Branch: refs/heads/master Commit: b50673ed8a36d3bb895954025d0a650c9b0aecab Parents: b4ae3fd Author: ggregory <ggreg...@apache.org> Authored: Thu Feb 11 21:27:36 2016 -0800 Committer: ggregory <ggreg...@apache.org> Committed: Thu Feb 11 21:27:36 2016 -0800 ---------------------------------------------------------------------- .../java/org/apache/logging/log4j/spi/AbstractLogger.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/b50673ed/log4j-api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java ---------------------------------------------------------------------- diff --git a/log4j-api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java b/log4j-api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java index aa69041..e7af6e9 100644 --- a/log4j-api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java +++ b/log4j-api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java @@ -553,12 +553,11 @@ public abstract class AbstractLogger implements ExtendedLogger, Serializable { return messageFactory.newMessage("exit"); } return messageFactory.newMessage("exit: " + format); - } else { - if (format == null) { - return messageFactory.newMessage("exit with(" + result + ')'); - } - return messageFactory.newMessage("exit: " + format, result); } + if (format == null) { + return messageFactory.newMessage("exit with(" + result + ')'); + } + return messageFactory.newMessage("exit: " + format, result); }