You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jmeter.apache.org by pm...@apache.org on 2017/02/18 21:40:47 UTC

svn commit: r1783592 - /jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/SocketOutputStreamPoolFactory.java

Author: pmouawad
Date: Sat Feb 18 21:40:47 2017
New Revision: 1783592

URL: http://svn.apache.org/viewvc?rev=1783592&view=rev
Log:
Comment on false positive

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/SocketOutputStreamPoolFactory.java

Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/SocketOutputStreamPoolFactory.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/SocketOutputStreamPoolFactory.java?rev=1783592&r1=1783591&r2=1783592&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/SocketOutputStreamPoolFactory.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/SocketOutputStreamPoolFactory.java Sat Feb 18 21:40:47 2017
@@ -71,7 +71,7 @@ public class SocketOutputStreamPoolFacto
     @Override
     public SocketOutputStream create(SocketConnectionInfos connectionInfos)
             throws Exception {
-        Socket socket = new Socket();
+        Socket socket = new Socket(); // NOSONAR closed by destroyObject
         socket.setKeepAlive(true);
         socket.setSoTimeout(socketTimeoutInMillis);
         socket.connect(new InetSocketAddress(connectionInfos.getHost(), connectionInfos.getPort()), socketConnectTimeoutInMillis);