Repository: logging-log4j2
Updated Branches:
  refs/heads/master 03c946363 -> 23235449d


Statement unnecessarily nested within 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/23235449
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/23235449
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/23235449

Branch: refs/heads/master
Commit: 23235449d5ddc776f4128c3ce082cb51bb9950a7
Parents: 03c9463
Author: ggregory <[email protected]>
Authored: Fri Jan 15 16:17:52 2016 -0800
Committer: ggregory <[email protected]>
Committed: Fri Jan 15 16:17:52 2016 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/logging/log4j/core/util/FileUtils.java   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/23235449/log4j-core/src/main/java/org/apache/logging/log4j/core/util/FileUtils.java
----------------------------------------------------------------------
diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/util/FileUtils.java 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/util/FileUtils.java
index 4d0e216..ad675ed 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/util/FileUtils.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/util/FileUtils.java
@@ -97,9 +97,8 @@ public final class FileUtils {
         String fileName = file.getName();
         if (fileName.lastIndexOf(".") != -1 && fileName.lastIndexOf(".") != 0) 
{
             return fileName.substring(fileName.lastIndexOf(".") + 1);
-        } else {
-            return null;
         }
+        return null;
     }
 
     /**

Reply via email to