You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by mi...@apache.org on 2016/08/30 09:27:21 UTC

[04/24] logging-log4j2 git commit: Use try-with-resources for JDBC resources.

Use try-with-resources for JDBC resources.

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

Branch: refs/heads/LOG4J2-1528
Commit: b6ecddc584fc0339dad323b4da69d6fc76c50b33
Parents: c4962aa
Author: Gary Gregory <gg...@apache.org>
Authored: Sun Aug 28 13:33:35 2016 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Sun Aug 28 13:33:35 2016 -0700

----------------------------------------------------------------------
 .../core/appender/db/jpa/AbstractJpaAppenderTest.java   | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/b6ecddc5/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/db/jpa/AbstractJpaAppenderTest.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/db/jpa/AbstractJpaAppenderTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/db/jpa/AbstractJpaAppenderTest.java
index 8fb2aaa..b150f1e 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/db/jpa/AbstractJpaAppenderTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/db/jpa/AbstractJpaAppenderTest.java
@@ -68,18 +68,8 @@ public abstract class AbstractJpaAppenderTest {
             context.reconfigure();
             StatusLogger.getLogger().reset();
 
-            Statement statement = null;
-            try {
-                statement = this.connection.createStatement();
+            try (Statement statement = this.connection.createStatement();) {
                 statement.execute("SHUTDOWN");
-            } finally {
-                try {
-                    if (statement != null) {
-                        statement.close();
-                    }
-                } catch (final SQLException ignore) {
-                    /* */
-                }
             }
 
             this.connection.close();