diff --git a/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java b/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java index b001e0a9c8bf..08023ff4201e 100644 --- a/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java +++ b/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java @@ -293,7 +293,7 @@ public abstract class ImportClassFixBase action.execute()); return Result.CLASS_AUTO_IMPORTED; @@ -349,15 +349,15 @@ public abstract class ImportClassFixBase(runnable, true); } - private void retrieveAndStoreNotificationInfo(@NotNull Runnable runnable) { + @NotNull + private static NotificationInfo getNotificationInfo(@NotNull Runnable runnable) { if (runnable instanceof ImportOptimizer.CollectingInfoRunnable) { String optimizerMessage = ((ImportOptimizer.CollectingInfoRunnable)runnable).getUserNotificationInfo(); - myOptimizerNotifications.add(optimizerMessage != null ? new NotificationInfo(optimizerMessage) : NOTHING_CHANGED_NOTIFICATION); + return optimizerMessage == null ? NOTHING_CHANGED_NOTIFICATION : new NotificationInfo(optimizerMessage); } - else if (runnable == EmptyRunnable.getInstance()) { - myOptimizerNotifications.add(NOTHING_CHANGED_NOTIFICATION); - } - else { - myOptimizerNotifications.add(SOMETHING_CHANGED_WITHOUT_MESSAGE_NOTIFICATION); + if (runnable == EmptyRunnable.getInstance()) { + return NOTHING_CHANGED_NOTIFICATION; } + return SOMETHING_CHANGED_WITHOUT_MESSAGE_NOTIFICATION; } private void putNotificationInfoIntoCollector() {