Repository: logging-log4j2 Updated Branches: refs/heads/master 58d611012 -> 6cb166400
Use final. Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/6cb16640 Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/6cb16640 Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/6cb16640 Branch: refs/heads/master Commit: 6cb166400fb22553af3b1aaa2ffd7bba6dd35ad5 Parents: 58d6110 Author: ggregory <[email protected]> Authored: Mon Jun 22 10:20:21 2015 -0700 Committer: ggregory <[email protected]> Committed: Mon Jun 22 10:20:21 2015 -0700 ---------------------------------------------------------------------- .../org/apache/logging/slf4j/MarkerTest.java | 26 ++++++++++---------- 1 file changed, 13 insertions(+), 13 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/6cb16640/log4j-slf4j-impl/src/test/java/org/apache/logging/slf4j/MarkerTest.java ---------------------------------------------------------------------- diff --git a/log4j-slf4j-impl/src/test/java/org/apache/logging/slf4j/MarkerTest.java b/log4j-slf4j-impl/src/test/java/org/apache/logging/slf4j/MarkerTest.java index 1897bd3..8c62cce 100644 --- a/log4j-slf4j-impl/src/test/java/org/apache/logging/slf4j/MarkerTest.java +++ b/log4j-slf4j-impl/src/test/java/org/apache/logging/slf4j/MarkerTest.java @@ -46,19 +46,19 @@ public class MarkerTest { slf4jMarker.add(slf4jParent); final Marker log4jParent = MarkerManager.getMarker(PARENT_MARKER_NAME + "-ANM"); final Marker log4jMarker = MarkerManager.getMarker(CHILD_MAKER_NAME + "ANM"); - Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); - Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); + final Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); + final Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); final org.slf4j.Marker nullMarker = null; try { log4jSlf4jParent.add(nullMarker); fail("Expected " + IllegalArgumentException.class.getName()); - } catch (IllegalArgumentException e) { + } catch (final IllegalArgumentException e) { // expected } try { log4jSlf4jMarker.add(nullMarker); fail("Expected " + IllegalArgumentException.class.getName()); - } catch (IllegalArgumentException e) { + } catch (final IllegalArgumentException e) { // expected } } @@ -70,19 +70,19 @@ public class MarkerTest { slf4jMarker.add(slf4jParent); final Marker log4jParent = MarkerManager.getMarker(PARENT_MARKER_NAME + "-CM"); final Marker log4jMarker = MarkerManager.getMarker(CHILD_MAKER_NAME + "-CM"); - Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); - Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); + final Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); + final Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); final org.slf4j.Marker nullMarker = null; try { Assert.assertFalse(log4jSlf4jParent.contains(nullMarker)); fail("Expected " + IllegalArgumentException.class.getName()); - } catch (IllegalArgumentException e) { + } catch (final IllegalArgumentException e) { // expected } try { Assert.assertFalse(log4jSlf4jMarker.contains(nullMarker)); fail("Expected " + IllegalArgumentException.class.getName()); - } catch (IllegalArgumentException e) { + } catch (final IllegalArgumentException e) { // expected } } @@ -94,9 +94,9 @@ public class MarkerTest { slf4jMarker.add(slf4jParent); final Marker log4jParent = MarkerManager.getMarker(PARENT_MARKER_NAME + "-CS"); final Marker log4jMarker = MarkerManager.getMarker(CHILD_MAKER_NAME + "-CS"); - Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); - Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); - String nullStr = null; + final Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); + final Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); + final String nullStr = null; Assert.assertFalse(log4jSlf4jParent.contains(nullStr)); Assert.assertFalse(log4jSlf4jMarker.contains(nullStr)); } @@ -123,8 +123,8 @@ public class MarkerTest { slf4jMarker.add(slf4jParent); final Marker log4jParent = MarkerManager.getMarker(PARENT_MARKER_NAME + "-CM"); final Marker log4jMarker = MarkerManager.getMarker(CHILD_MAKER_NAME + "-CM"); - Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); - Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); + final Log4jMarker log4jSlf4jParent = new Log4jMarker(log4jParent); + final Log4jMarker log4jSlf4jMarker = new Log4jMarker(log4jMarker); final org.slf4j.Marker nullMarker = null; Assert.assertFalse(log4jSlf4jParent.remove(nullMarker)); Assert.assertFalse(log4jSlf4jMarker.remove(nullMarker));
