This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/release-2.x by this push:
     new 55c6bb1  No need to nest in else clause.
55c6bb1 is described below

commit 55c6bb195450307689edd359ec55e4d332a2064a
Author: Gary Gregory <[email protected]>
AuthorDate: Thu Nov 29 18:42:21 2018 -0700

    No need to nest in else clause.
---
 .../apache/logging/log4j/core/appender/rolling/RollingFileManager.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
index 4a99e53..53b9b1d 100644
--- 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
+++ 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
@@ -670,9 +670,8 @@ public class RollingFileManager extends FileManager {
                 FileTime fileTime = attrs.creationTime();
                 if (fileTime.compareTo(EPOCH) > 0) {
                     return fileTime.toMillis();
-                } else {
-                    LOGGER.info("Unable to obtain file creation time for " + 
file.getAbsolutePath());
                 }
+                LOGGER.info("Unable to obtain file creation time for " + 
file.getAbsolutePath());
             } catch (Exception ex) {
                 LOGGER.info("Unable to calculate file creation time for " + 
file.getAbsolutePath() + ": " + ex.getMessage());
             }

Reply via email to