You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2017/06/29 20:28:42 UTC

logging-log4j2 git commit: Add comment and @SuppressWarning.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master df34929c8 -> 3caaba0d4


Add comment and @SuppressWarning.

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

Branch: refs/heads/master
Commit: 3caaba0d4fb21beff87e7626971756627a994644
Parents: df34929
Author: Gary Gregory <gg...@apache.org>
Authored: Thu Jun 29 13:28:38 2017 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Thu Jun 29 13:28:38 2017 -0700

----------------------------------------------------------------------
 .../java/org/apache/logging/log4j/core/net/TcpSocketManager.java    | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/3caaba0d/log4j-core/src/main/java/org/apache/logging/log4j/core/net/TcpSocketManager.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/net/TcpSocketManager.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/net/TcpSocketManager.java
index 50cf767..481f59f 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/net/TcpSocketManager.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/net/TcpSocketManager.java
@@ -307,6 +307,7 @@ public class TcpSocketManager extends AbstractSocketManager {
                 try {
                     sleep(reconnectionDelay);
                     final Socket sock = createSocket(inetAddress, port);
+                    @SuppressWarnings("resource") // newOS is managed by the enclosing Manager.
                     final OutputStream newOS = sock.getOutputStream();
                     synchronized (owner) {
                         try {