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

Posted to commits@jmeter.apache.org by se...@apache.org on 2013/08/27 21:46:19 UTC

svn commit: r1517954 - /jmeter/trunk/src/jorphan/org/apache/jorphan/exec/SystemCommand.java

Author: sebb
Date: Tue Aug 27 19:46:19 2013
New Revision: 1517954

URL: http://svn.apache.org/r1517954
Log:
Add comment re proc.destroy

Modified:
    jmeter/trunk/src/jorphan/org/apache/jorphan/exec/SystemCommand.java

Modified: jmeter/trunk/src/jorphan/org/apache/jorphan/exec/SystemCommand.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/jorphan/org/apache/jorphan/exec/SystemCommand.java?rev=1517954&r1=1517953&r2=1517954&view=diff
==============================================================================
--- jmeter/trunk/src/jorphan/org/apache/jorphan/exec/SystemCommand.java (original)
+++ jmeter/trunk/src/jorphan/org/apache/jorphan/exec/SystemCommand.java Tue Aug 27 19:46:19 2013
@@ -162,6 +162,7 @@ public class SystemCommand {
             }
             
             if (isAlive(proc)) {
+                // N.B. proc.destroy() is called by the finally clause in the run() method
                 throw new InterruptedException( "Process timeout out after " + timeoutInMillis + " milliseconds" );
             }
             return proc.exitValue();