diff --git a/idea/idea-analysis/src/org/jetbrains/jet/plugin/quickfix/JetIntentionActionsFactory.kt b/idea/idea-analysis/src/org/jetbrains/jet/plugin/quickfix/JetIntentionActionsFactory.kt index e6baa65717e..2728eef23bd 100644 --- a/idea/idea-analysis/src/org/jetbrains/jet/plugin/quickfix/JetIntentionActionsFactory.kt +++ b/idea/idea-analysis/src/org/jetbrains/jet/plugin/quickfix/JetIntentionActionsFactory.kt @@ -14,16 +14,11 @@ * limitations under the License. */ -package org.jetbrains.jet.plugin.quickfix; +package org.jetbrains.jet.plugin.quickfix -import com.intellij.codeInsight.intention.IntentionAction; -import org.jetbrains.annotations.NotNull; -import org.jetbrains.jet.lang.diagnostics.Diagnostic; +import com.intellij.codeInsight.intention.IntentionAction +import org.jetbrains.jet.lang.diagnostics.Diagnostic -import java.util.List; - -public interface JetIntentionActionsFactory { - - @NotNull - List createActions(@NotNull Diagnostic diagnostic); +public trait JetIntentionActionsFactory { + public fun createActions(diagnostic: Diagnostic): List } diff --git a/idea/src/org/jetbrains/jet/plugin/quickfix/JetSingleIntentionActionFactory.kt b/idea/src/org/jetbrains/jet/plugin/quickfix/JetSingleIntentionActionFactory.kt index de9ccdd58d9..230903f9321 100644 --- a/idea/src/org/jetbrains/jet/plugin/quickfix/JetSingleIntentionActionFactory.kt +++ b/idea/src/org/jetbrains/jet/plugin/quickfix/JetSingleIntentionActionFactory.kt @@ -14,39 +14,24 @@ * limitations under the License. */ -package org.jetbrains.jet.plugin.quickfix; +package org.jetbrains.jet.plugin.quickfix -import com.intellij.codeInsight.intention.IntentionAction; -import org.jetbrains.annotations.NotNull; -import org.jetbrains.annotations.Nullable; -import org.jetbrains.jet.lang.diagnostics.Diagnostic; -import org.jetbrains.jet.lang.psi.JetCodeFragment; +import com.intellij.codeInsight.intention.IntentionAction +import org.jetbrains.jet.lang.diagnostics.Diagnostic +import org.jetbrains.jet.lang.psi.JetCodeFragment -import java.util.LinkedList; -import java.util.List; +import java.util.Collections +import org.jetbrains.jet.utils.addToStdlib.singletonOrEmptyList -public abstract class JetSingleIntentionActionFactory implements JetIntentionActionsFactory { +public abstract class JetSingleIntentionActionFactory : JetIntentionActionsFactory { + public abstract fun createAction(diagnostic: Diagnostic): IntentionAction? - @Nullable - public abstract IntentionAction createAction(@NotNull Diagnostic diagnostic); - - @NotNull - @Override - public final List createActions(@NotNull Diagnostic diagnostic) { - List intentionActionList = new LinkedList(); - - if (diagnostic.getPsiElement().getContainingFile() instanceof JetCodeFragment && !isApplicableForCodeFragment()) { - return intentionActionList; + override fun createActions(diagnostic: Diagnostic): List { + if (diagnostic.getPsiElement().getContainingFile() is JetCodeFragment && !isApplicableForCodeFragment()) { + return Collections.emptyList() } - - IntentionAction intentionAction = createAction(diagnostic); - if (intentionAction != null) { - intentionActionList.add(intentionAction); - } - return intentionActionList; + return createAction(diagnostic).singletonOrEmptyList() } - public boolean isApplicableForCodeFragment() { - return false; - } + public fun isApplicableForCodeFragment(): Boolean = false }