diff --git a/src/org/jetbrains/jet/j2k/ast/Enum.kt b/src/org/jetbrains/jet/j2k/ast/Enum.kt index fef0b38cc7f..df72fee1414 100644 --- a/src/org/jetbrains/jet/j2k/ast/Enum.kt +++ b/src/org/jetbrains/jet/j2k/ast/Enum.kt @@ -28,7 +28,7 @@ public open class Enum(converter : Converter, primaryConstructorSignatureToKotlin() + typeParametersToKotlin() + implementTypesToKotlin() + - " {\n" + AstUtil.joinNodes(membersExceptConstructors(), "\n") + "\n" + + " {\n" + membersExceptConstructors().toKotlin("\n") + "\n" + primaryConstructorBodyToKotlin() + "\npublic fun name() : String { return \"\" }\npublic fun order() : Int { return 0 }\n}" } diff --git a/src/org/jetbrains/jet/j2k/ast/File.kt b/src/org/jetbrains/jet/j2k/ast/File.kt index 52b9bfff84c..960fcc22234 100644 --- a/src/org/jetbrains/jet/j2k/ast/File.kt +++ b/src/org/jetbrains/jet/j2k/ast/File.kt @@ -9,7 +9,7 @@ public open class File(val packageName: String, val mainFunction: String): Node() { public override fun toKotlin(): String { - val common: String = AstUtil.joinNodes(imports, "\n") + "\n\n" + AstUtil.joinNodes(classes, "\n") + "\n" + mainFunction + val common: String = imports.toKotlin("\n") + "\n\n" + classes.toKotlin("\n") + "\n" + mainFunction if (packageName.isEmpty()) { return common } diff --git a/src/org/jetbrains/jet/j2k/ast/MethodCallExpression.kt b/src/org/jetbrains/jet/j2k/ast/MethodCallExpression.kt index a382080b2c3..ee61f3f72de 100644 --- a/src/org/jetbrains/jet/j2k/ast/MethodCallExpression.kt +++ b/src/org/jetbrains/jet/j2k/ast/MethodCallExpression.kt @@ -11,10 +11,7 @@ public open class MethodCallExpression(val methodCall: Expression, public override fun isNullable(): Boolean = methodCall.isNullable() || resultIsNullable public override fun toKotlin(): String { - val typeParamsToKotlin: String? = (if (typeParameters.size() > 0) - "<" + AstUtil.joinNodes(typeParameters, ", ") + ">" - else - "") + val typeParamsToKotlin: String = typeParameters.toKotlin(", ", "<", ">") val argumentsMapped = arguments.map { it.toKotlin() } return methodCall.toKotlin() + typeParamsToKotlin + "(" + argumentsMapped.makeString(", ") + ")" } diff --git a/src/org/jetbrains/jet/j2k/ast/NewClassExpression.java b/src/org/jetbrains/jet/j2k/ast/NewClassExpression.java index 038a158ea99..e0d93b812b9 100644 --- a/src/org/jetbrains/jet/j2k/ast/NewClassExpression.java +++ b/src/org/jetbrains/jet/j2k/ast/NewClassExpression.java @@ -38,7 +38,7 @@ public class NewClassExpression extends Expression { myArguments = arguments; } - public NewClassExpression(@NotNull Expression qualifier, @NotNull Element name, @NotNull List arguments, + public NewClassExpression(@NotNull Element name, @NotNull List arguments, @NotNull Expression qualifier, @Nullable AnonymousClass anonymousClass) { this(name, arguments); myQualifier = qualifier; diff --git a/src/org/jetbrains/jet/j2k/ast/ReferenceElement.kt b/src/org/jetbrains/jet/j2k/ast/ReferenceElement.kt index ffb66d26fe4..79d4e97f283 100644 --- a/src/org/jetbrains/jet/j2k/ast/ReferenceElement.kt +++ b/src/org/jetbrains/jet/j2k/ast/ReferenceElement.kt @@ -5,11 +5,5 @@ import org.jetbrains.jet.j2k.util.AstUtil import java.util.List public open class ReferenceElement(val reference : Identifier, val types : List) : Element() { - public override fun toKotlin() : String { - val typesIfNeeded : String = (if (types.size() > 0) - "<" + AstUtil.joinNodes(types, ", ") + ">" - else - "") - return reference.toKotlin() + typesIfNeeded - } + public override fun toKotlin() = reference.toKotlin() + types.toKotlin(", ", "<", ">") } diff --git a/src/org/jetbrains/jet/j2k/visitors/ExpressionVisitor.java b/src/org/jetbrains/jet/j2k/visitors/ExpressionVisitor.java index 3a64e069b7d..c8cf0b5b263 100644 --- a/src/org/jetbrains/jet/j2k/visitors/ExpressionVisitor.java +++ b/src/org/jetbrains/jet/j2k/visitors/ExpressionVisitor.java @@ -277,9 +277,7 @@ public class ExpressionVisitor extends StatementVisitor { PsiExpression[] arguments = argumentList != null ? argumentList.getExpressions() : new PsiExpression[]{}; if (constructor == null || Converter.$classobj.isConstructorPrimary(constructor) || isNotConvertedClass) { return new NewClassExpression( - getConverter().expressionToExpression(expression.getQualifier()), - getConverter().elementToElement(classReference), - getConverter().argumentsToExpressionList(expression), + getConverter().elementToElement(classReference), getConverter().argumentsToExpressionList(expression), getConverter().expressionToExpression(expression.getQualifier()), anonymousClass != null ? getConverter().anonymousClassToAnonymousClass(anonymousClass) : null ); }