From: Eugene Zhuravlev <jeka@intellij.com>
Date: Mon, 30 Jan 2012 15:53:52 +0000 (+0100)
Subject: automake: notifications about successful compilations seem to be redundant
X-Git-Tag: rubymine/112.645~11^2~5^2
X-Git-Url: https://git.jetbrains.org/?p=idea%2Fcommunity.git;a=commitdiff_plain;h=dc2e864253bf18427d2361406aa1b35ad5f26bee

automake: notifications about successful compilations seem to be redundant
---

diff --git a/java/compiler/impl/src/com/intellij/compiler/CompileServerManager.java b/java/compiler/impl/src/com/intellij/compiler/CompileServerManager.java
index 6a6ef6bc66af..9e0e25ca9c94 100644
--- a/java/compiler/impl/src/com/intellij/compiler/CompileServerManager.java
+++ b/java/compiler/impl/src/com/intellij/compiler/CompileServerManager.java
@@ -695,13 +695,13 @@ public class CompileServerManager implements ApplicationComponent{
 
     @Override
     public void sessionTerminated() {
-      String statusMessage = "Auto make completed";
+      String statusMessage = null/*"Auto make completed"*/;
       switch (myBuildStatus) {
         case SUCCESS:
-          statusMessage = "Auto make completed successfully";
+          //statusMessage = "Auto make completed successfully";
           break;
         case UP_TO_DATE:
-          statusMessage = "All files are up-to-date";
+          //statusMessage = "All files are up-to-date";
           break;
         case ERRORS:
           statusMessage = "Auto make completed with errors";
@@ -710,9 +710,11 @@ public class CompileServerManager implements ApplicationComponent{
           statusMessage = "Auto make has been canceled";
           break;
       }
-      final Notification notification = CompilerManager.NOTIFICATION_GROUP.createNotification(statusMessage, MessageType.INFO);
-      if (!myProject.isDisposed()) {
-        notification.notify(myProject);
+      if (statusMessage != null) {
+        final Notification notification = CompilerManager.NOTIFICATION_GROUP.createNotification(statusMessage, MessageType.INFO);
+        if (!myProject.isDisposed()) {
+          notification.notify(myProject);
+        }
       }
     }