From 1901424bdbf6f475375b1024983a1b41d6b03faa Mon Sep 17 00:00:00 2001 From: "Natalia.Ukhorskaya" Date: Wed, 20 Feb 2013 15:22:13 +0400 Subject: [PATCH] Move method: defaultInitializer GitOrigin-RevId: 4a3365ef2ac8be58aa94d2b650cc762781728539 --- .../plugin/codeInsight/CodeInsightUtils.java | 20 +++++++++++++++++++ .../OverrideImplementMethodsHandler.java | 18 +---------------- 2 files changed, 21 insertions(+), 17 deletions(-) diff --git a/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/CodeInsightUtils.java b/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/CodeInsightUtils.java index 157820cf3b0b..97077608080b 100644 --- a/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/CodeInsightUtils.java +++ b/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/CodeInsightUtils.java @@ -12,6 +12,8 @@ import org.jetbrains.jet.lang.psi.JetBlockExpression; import org.jetbrains.jet.lang.psi.JetElement; import org.jetbrains.jet.lang.psi.JetExpression; import org.jetbrains.jet.lang.psi.JetFile; +import org.jetbrains.jet.lang.types.JetType; +import org.jetbrains.jet.lang.types.lang.KotlinBuiltIns; import java.util.ArrayList; import java.util.List; @@ -132,6 +134,24 @@ public class CodeInsightUtils { return element; } + public static String defaultInitializer(JetType type) { + KotlinBuiltIns builtIns = KotlinBuiltIns.getInstance(); + if (type.isNullable()) { + return "null"; + } + else if (type.equals(builtIns.getIntType()) || type.equals(builtIns.getLongType()) || + type.equals(builtIns.getShortType()) || type.equals(builtIns.getByteType()) || + type.equals(builtIns.getFloatType()) || type.equals(builtIns.getDoubleType()) || + type.equals(builtIns.getCharType())) { + return "0"; + } + else if (type.equals(builtIns.getBooleanType())) { + return "false"; + } + + return null; + } + private CodeInsightUtils() { } } diff --git a/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/OverrideImplementMethodsHandler.java b/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/OverrideImplementMethodsHandler.java index 882f32850400..f637cc09325a 100644 --- a/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/OverrideImplementMethodsHandler.java +++ b/plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/OverrideImplementMethodsHandler.java @@ -141,7 +141,7 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns bodyBuilder.append(descriptor.getName()).append(" : ").append( DescriptorRenderer.SHORT_NAMES_IN_TYPES.renderType(descriptor.getType())); - String initializer = defaultInitializer(descriptor.getType(), KotlinBuiltIns.getInstance()); + String initializer = CodeInsightUtils.defaultInitializer(descriptor.getType()); if (initializer != null) { bodyBuilder.append(" = ").append(initializer); } @@ -247,22 +247,6 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns } } - private static String defaultInitializer(JetType returnType, KotlinBuiltIns builtIns) { - if (returnType.isNullable()) { - return "null"; - } - else if (returnType.equals(builtIns.getIntType()) || returnType.equals(builtIns.getLongType()) || - returnType.equals(builtIns.getShortType()) || returnType.equals(builtIns.getByteType()) || - returnType.equals(builtIns.getFloatType()) || returnType.equals(builtIns.getDoubleType())) { - return "0"; - } - else if (returnType.equals(builtIns.getBooleanType())) { - return "false"; - } - - return null; - } - private static String displayableVisibility(MemberDescriptor descriptor) { Visibility visibility = descriptor.getVisibility().normalize(); return visibility != Visibilities.INTERNAL ? visibility.toString() + " " : "";