You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by rp...@apache.org on 2016/08/12 12:11:52 UTC

[06/50] logging-log4j2 git commit: Comments.

Comments.

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/e81c6a11
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/e81c6a11
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/e81c6a11

Branch: refs/heads/LOG4J2-1010&LOG4J2-1447-injectable-contextdata&better-datastructure
Commit: e81c6a11d5c5f07c83ad81fc47cbb94f2cbbfc2b
Parents: 96f7d65
Author: Gary Gregory <gg...@apache.org>
Authored: Sun Aug 7 11:27:56 2016 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Sun Aug 7 11:27:56 2016 -0700

----------------------------------------------------------------------
 .../test/java/org/apache/logging/log4j/junit/CleanFolders.java    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/e81c6a11/log4j-core/src/test/java/org/apache/logging/log4j/junit/CleanFolders.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/junit/CleanFolders.java b/log4j-core/src/test/java/org/apache/logging/log4j/junit/CleanFolders.java
index a4d9023..ea75a55 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/junit/CleanFolders.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/junit/CleanFolders.java
@@ -53,7 +53,7 @@ public class CleanFolders extends AbstractExternalFileCleaner {
     @Override
     protected void clean() {
         Map<Path, IOException> failures = new HashMap<>();
-
+        // Clean and gather failures
         for (final File folder : getFiles()) {
             if (folder.exists()) {
                 final Path path = folder.toPath();
@@ -72,6 +72,7 @@ public class CleanFolders extends AbstractExternalFileCleaner {
                 }
             }
         }
+        // Fail on failures
         if (failures.size() > 0) {
             StringBuilder sb = new StringBuilder();
             boolean first = true;