Replace try finally statement with try with resources statement (#510)
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/EventParameterMemoryLeakTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/EventParameterMemoryLeakTest.java
index 944e987..b7a28bc 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/EventParameterMemoryLeakTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/EventParameterMemoryLeakTest.java
@@ -73,12 +73,9 @@
assertThat(line3, containsString("paramValue"));
assertThat(line4, containsString("paramValue"));
assertNull(line5, "Expected only three lines");
- GarbageCollectionHelper gcHelper = new GarbageCollectionHelper();
- gcHelper.run();
- try {
+ try (GarbageCollectionHelper gcHelper = new GarbageCollectionHelper()) {
+ gcHelper.run();
assertTrue(latch.await(30, TimeUnit.SECONDS), "Parameter should have been garbage collected");
- } finally {
- gcHelper.close();
}
}
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/ReusableParameterizedMessageMemoryLeakTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/ReusableParameterizedMessageMemoryLeakTest.java
index b162d27..93081ee 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/ReusableParameterizedMessageMemoryLeakTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/ReusableParameterizedMessageMemoryLeakTest.java
@@ -34,12 +34,9 @@
"foo {}", new ParameterObject("paramValue", latch));
// Large enough for the parameters, but smaller than the default reusable array size.
message.swapParameters(new Object[5]);
- GarbageCollectionHelper gcHelper = new GarbageCollectionHelper();
- gcHelper.run();
- try {
+ try (GarbageCollectionHelper gcHelper = new GarbageCollectionHelper()) {
+ gcHelper.run();
assertTrue(latch.await(30, TimeUnit.SECONDS), "Parameter should have been garbage collected");
- } finally {
- gcHelper.close();
}
}
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncAppenderConfigTest_LOG4J2_2032.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncAppenderConfigTest_LOG4J2_2032.java
index f1e7d64..f0e1685 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncAppenderConfigTest_LOG4J2_2032.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncAppenderConfigTest_LOG4J2_2032.java
@@ -50,13 +50,11 @@
log.info("Text containing curly braces: {}", "Curly{}");
CoreLoggerContexts.stopLoggerContext(file); // stop async thread
- final BufferedReader reader = new BufferedReader(new FileReader(file));
- try {
+ try (BufferedReader reader = new BufferedReader(new FileReader(file))) {
final String line1 = reader.readLine();
System.out.println(line1);
assertTrue("line1 correct", line1.contains(" Text containing curly braces: Curly{} "));
} finally {
- reader.close();
file.delete();
}
}
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncLoggerTestArgumentFreedOnError.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncLoggerTestArgumentFreedOnError.java
index 3d767fd..6c1487a 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncLoggerTestArgumentFreedOnError.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/async/AsyncLoggerTestArgumentFreedOnError.java
@@ -57,13 +57,10 @@
final AsyncLogger log = (AsyncLogger) LogManager.getLogger("com.foo.Bar");
CountDownLatch garbageCollectionLatch = new CountDownLatch(1);
log.fatal(new ThrowingMessage(garbageCollectionLatch));
- GarbageCollectionHelper gcHelper = new GarbageCollectionHelper();
- gcHelper.run();
- try {
+ try (GarbageCollectionHelper gcHelper = new GarbageCollectionHelper()) {
+ gcHelper.run();
assertTrue("Parameter should have been garbage collected",
garbageCollectionLatch.await(30, TimeUnit.SECONDS));
- } finally {
- gcHelper.close();
}
}
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/async/QueueFullAsyncLoggerTest3.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/async/QueueFullAsyncLoggerTest3.java
index 198cb6b..5813bec 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/async/QueueFullAsyncLoggerTest3.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/async/QueueFullAsyncLoggerTest3.java
@@ -85,12 +85,9 @@
}
blockingAppender.countDownLatch.countDown();
- final GarbageCollectionHelper gcHelper = new GarbageCollectionHelper();
- gcHelper.run();
- try {
+ try (GarbageCollectionHelper gcHelper = new GarbageCollectionHelper()) {
+ gcHelper.run();
assertTrue("Parameter should have been garbage collected", garbageCollectionLatch.await(30, TimeUnit.SECONDS));
- } finally {
- gcHelper.close();
}
}
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/config/NestedLoggerConfigTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/config/NestedLoggerConfigTest.java
index fb14bdf..edec79c 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/config/NestedLoggerConfigTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/config/NestedLoggerConfigTest.java
@@ -68,14 +68,11 @@
}
private Configuration loadConfiguration(String resourcePath) throws IOException {
- InputStream in = getClass().getClassLoader().getResourceAsStream(resourcePath);
- try {
+ try (InputStream in = getClass().getClassLoader().getResourceAsStream(resourcePath)) {
Configuration configuration = new XmlConfiguration(new LoggerContext("test"), new ConfigurationSource(in));
configuration.initialize();
configuration.start();
return configuration;
- } finally {
- in.close();
}
}
}
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/impl/NestedLoggingFromThrowableMessageTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/impl/NestedLoggingFromThrowableMessageTest.java
index 5f58af7..dd4a5c5 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/impl/NestedLoggingFromThrowableMessageTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/impl/NestedLoggingFromThrowableMessageTest.java
@@ -87,14 +87,11 @@
private static Set<String> readUniqueLines(File input) throws IOException {
Set<String> lines = new HashSet<>();
- BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(input)));
- try {
+ try (BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(input)))) {
String line;
while ((line = reader.readLine()) != null) {
assertTrue("Read duplicate line: " + line, lines.add(line));
}
- } finally {
- reader.close();
}
return lines;
}