Merge remote-tracking branch 'origin/amakeev/cmake-configs' into amakeev/cmake-config... clion/171.256
authorAnton Makeev <Anton.Makeev@jetbrains.com>
Wed, 19 Oct 2016 22:40:23 +0000 (00:40 +0200)
committerAnton Makeev <Anton.Makeev@jetbrains.com>
Wed, 19 Oct 2016 22:40:23 +0000 (00:40 +0200)
platform/platform-api/src/com/intellij/execution/process/CapturingProcessHandler.java
platform/platform-api/src/com/intellij/execution/process/ProcessOutput.java
platform/platform-impl/src/com/intellij/util/ContentUtilEx.java
spellchecker/src/com/intellij/spellchecker/jetbrains.dic

index dc3245de185caa29e71e86d3bb9a24301d4a735d..26e7b83e47f40c596f65977833f344b0eb5ffb28 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.hasErrorExitCode()) {
+    if (!myOutput.isErrorCodeSet()) {
       myOutput.setExitCode(getProcess().exitValue());
     }
   }
index 3f7fd092494382afb8f6ea621f586cdaab36086d..b8b849e332e2f049d5b7445635f28b2ad02c7e9a 100644 (file)
@@ -26,16 +26,13 @@ import java.util.List;
  * @author yole
  */
 public class ProcessOutput {
-  private static final int ERROR_EXIT_CODE = -1;
-  
   private final StringBuilder myStdoutBuilder = new StringBuilder();
   private final StringBuilder myStderrBuilder = new StringBuilder();
-  private int myExitCode;
+  @Nullable private Integer myExitCode;
   private boolean myTimeout;
   private boolean myCancelled;
 
   public ProcessOutput() {
-    myExitCode = ERROR_EXIT_CODE; // until set explicitly, exit code denotes an error.
   }
 
   public ProcessOutput(final int exitCode) {
@@ -106,15 +103,16 @@ public class ProcessOutput {
   }
 
   public int getExitCode() {
-    return myExitCode;
+    Integer code = myExitCode;
+    return code == null ? -1 : code;
   }
 
   /**
-   * @return true if exit code wasn't set and is still set to default value (this might happen
+   * @return false if exit code wasn't set
    * for example, when our CapturingProcessHandler.runProcess() is interrupted)
    */
-  public boolean hasErrorExitCode() {
-    return getExitCode() == ERROR_EXIT_CODE;
+  public boolean isErrorCodeSet() {
+    return myExitCode != null;
   }
 
   public void setTimeout() {
index 1ef90b233c57e12c92058564ced9513c2a5cc8cc..52f6ea2d0e2ed51a794797202ecd1d5442a408d9 100644 (file)
@@ -21,6 +21,7 @@ import com.intellij.openapi.util.Comparing;
 import com.intellij.openapi.util.Condition;
 import com.intellij.openapi.util.Disposer;
 import com.intellij.openapi.util.Pair;
+import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.ui.content.Content;
 import com.intellij.ui.content.ContentFactory;
 import com.intellij.ui.content.ContentManager;
@@ -198,10 +199,7 @@ public class ContentUtilEx extends ContentsUtil {
 
   @Nullable
   public static String getTabNameWithoutPrefix(@NotNull TabbedContent content, @NotNull String fullTabName) {
-    int fullPrefixLength = getFullPrefix(content.getTitlePrefix()).length();
-    if (fullTabName.startsWith(content.getTitlePrefix())) {
-      return fullTabName.substring(fullPrefixLength);
-    }
-    return null;
+    String prefix = content.getTitlePrefix();
+    return prefix == null ? fullTabName : StringUtil.trimStart(fullTabName, getFullPrefix(prefix));
   }
 }
index 27dcb0c0d9107bb1955819f91aa0e0eda2deb391..5d21d75c3786f7ca305fe1e0188d7038282a15d5 100644 (file)
@@ -319,6 +319,7 @@ lvalue
 lvalues
 macrodef
 makefile
+makefiles
 maxdatafiles
 maxextents
 maximizable