Confusing and incorrect ProcessOutput API fixed.
authorAnton Makeev <Anton.Makeev@jetbrains.com>
Thu, 20 Oct 2016 15:55:53 +0000 (17:55 +0200)
committerAnton Makeev <Anton.Makeev@jetbrains.com>
Thu, 20 Oct 2016 15:55:53 +0000 (17:55 +0200)
platform/platform-api/src/com/intellij/execution/process/CapturingProcessHandler.java
platform/platform-api/src/com/intellij/execution/process/ProcessOutput.java

index 26e7b83e47f40c596f65977833f344b0eb5ffb28..7e3b67142f79ba3ac3404bcd99042934fb56d57b 100644 (file)
@@ -78,7 +78,7 @@ public class CapturingProcessHandler extends OSProcessHandler {
   private void setErrorCodeIfNotYetSet() {
     // if exit code was set on processTerminated, no need to rewrite it
     // WinPtyProcess returns -2 if pty is already closed
-    if (!myOutput.isErrorCodeSet()) {
+    if (!myOutput.isExitCodeSet()) {
       myOutput.setExitCode(getProcess().exitValue());
     }
   }
index b8b849e332e2f049d5b7445635f28b2ad02c7e9a..be0cb586836727a78cefd7ef447cc7ed8da29336 100644 (file)
@@ -111,7 +111,7 @@ public class ProcessOutput {
    * @return false if exit code wasn't set, 
    * for example, when our CapturingProcessHandler.runProcess() is interrupted)
    */
-  public boolean isErrorCodeSet() {
+  public boolean isExitCodeSet() {
     return myExitCode != null;
   }