Add comments back

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

Branch: refs/heads/master
Commit: 65706c0e4b7d08c430e62d8cf5cc7dc4b4f77368
Parents: bb8f2ee
Author: Mikael StÃ¥ldal <[email protected]>
Authored: Sat Jul 8 21:38:14 2017 +0200
Committer: Mikael StÃ¥ldal <[email protected]>
Committed: Sat Jul 8 21:38:14 2017 +0200

----------------------------------------------------------------------
 .../org/apache/logging/log4j/ThreadContextUtilityClass.java | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/65706c0e/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java
----------------------------------------------------------------------
diff --git 
a/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java
 
b/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java
index 21a9951..a5603c5 100644
--- 
a/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java
+++ 
b/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java
@@ -53,15 +53,20 @@ public class ThreadContextUtilityClass {
         ThreadContext.clearMap();
         final Map<String, String> map1 = ThreadContext.getContext();
         assertTrue(map1.isEmpty());
-        map1.put("K", "val");
+        map1.put("K", "val"); // no error
         assertEquals("val", map1.get("K"));
+
+        // adding to copy does not affect thread context map
         assertTrue(ThreadContext.getContext().isEmpty());
+
         ThreadContext.put("key", "val2");
         final Map<String, String> map2 = ThreadContext.getContext();
         assertEquals(1, map2.size());
         assertEquals("val2", map2.get("key"));
-        map2.put("K", "val");
+        map2.put("K", "val"); // no error
         assertEquals("val", map2.get("K"));
+
+        // first copy is not affected
         assertNotSame(map1, map2);
         assertEquals(1, map1.size());
     }

Reply via email to