diff --git a/idea/src/org/jetbrains/kotlin/idea/intentions/MoveLambdaOutsideParenthesesIntention.kt b/idea/src/org/jetbrains/kotlin/idea/intentions/MoveLambdaOutsideParenthesesIntention.kt index 06923854d19..47865612211 100644 --- a/idea/src/org/jetbrains/kotlin/idea/intentions/MoveLambdaOutsideParenthesesIntention.kt +++ b/idea/src/org/jetbrains/kotlin/idea/intentions/MoveLambdaOutsideParenthesesIntention.kt @@ -59,4 +59,8 @@ public class MoveLambdaOutsideParenthesesIntention : JetSelfTargetingIntention, val typeArguments: List, - override val isNullable: Boolean, - val lambdaArgument: LambdaExpression? = null + override val isNullable: Boolean ) : Expression() { override fun generateCode(builder: CodeBuilder) { builder.appendOperand(this, methodExpression).append(typeArguments, ", ", "<", ">") - if (arguments.isNotEmpty() || lambdaArgument == null) { - builder.append("(").append(arguments, ", ").append(")") - } - if (lambdaArgument != null) { - builder.append(lambdaArgument) - } + builder.append("(").append(arguments, ", ").append(")") } companion object { public fun buildNotNull(receiver: Expression?, methodName: String, arguments: List = listOf(), - typeArguments: List = listOf(), - lambdaArgument: LambdaExpression? = null): MethodCallExpression - = build(receiver, methodName, arguments, typeArguments, false, lambdaArgument) + typeArguments: List = listOf()): MethodCallExpression + = build(receiver, methodName, arguments, typeArguments, false) public fun buildNullable(receiver: Expression?, methodName: String, arguments: List = listOf(), - typeArguments: List = listOf(), - lambdaArgument: LambdaExpression? = null): MethodCallExpression - = build(receiver, methodName, arguments, typeArguments, true, lambdaArgument) + typeArguments: List = listOf()): MethodCallExpression + = build(receiver, methodName, arguments, typeArguments, true) public fun build(receiver: Expression?, methodName: String, arguments: List, typeArguments: List, - isNullable: Boolean, - lambdaArgument: LambdaExpression? = null): MethodCallExpression { + isNullable: Boolean): MethodCallExpression { val identifier = Identifier(methodName, false).assignNoPrototype() return MethodCallExpression(if (receiver != null) QualifiedExpression(receiver, identifier).assignNoPrototype() else identifier, arguments, typeArguments, - isNullable, - lambdaArgument) + isNullable) } } } diff --git a/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimension3d.kt b/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimension3d.kt index f4cc76738ab..e9b281967ef 100644 --- a/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimension3d.kt +++ b/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimension3d.kt @@ -1 +1 @@ -val d3 = Array>(5, { arrayOfNulls(5) }) \ No newline at end of file +val d3 = Array>(5) { arrayOfNulls(5) } \ No newline at end of file diff --git a/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimensionExplicit.kt b/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimensionExplicit.kt index 8bfd2228d30..1c8a8659a40 100644 --- a/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimensionExplicit.kt +++ b/j2k/testData/fileOrElement/arrayType/arrayInitializationStatementWithDimensionExplicit.kt @@ -1 +1 @@ -val d2 = Array(5, { IntArray(5) }) \ No newline at end of file +val d2 = Array(5) { IntArray(5) } \ No newline at end of file diff --git a/j2k/testData/fileOrElement/arrayType/d2StringEmptyArray.kt b/j2k/testData/fileOrElement/arrayType/d2StringEmptyArray.kt index 52d2dcfe14c..8180c6e2c7e 100644 --- a/j2k/testData/fileOrElement/arrayType/d2StringEmptyArray.kt +++ b/j2k/testData/fileOrElement/arrayType/d2StringEmptyArray.kt @@ -1 +1 @@ -val ss = Array>(5, { arrayOfNulls(5) }) \ No newline at end of file +val ss = Array>(5) { arrayOfNulls(5) } \ No newline at end of file diff --git a/j2k/testData/fileOrElement/arrayType/d3StringEmptyArray.kt b/j2k/testData/fileOrElement/arrayType/d3StringEmptyArray.kt index 7f2164c3d1f..e2fe7c4cc84 100644 --- a/j2k/testData/fileOrElement/arrayType/d3StringEmptyArray.kt +++ b/j2k/testData/fileOrElement/arrayType/d3StringEmptyArray.kt @@ -1 +1 @@ -val sss = Array(5, { Array>(5, { arrayOfNulls(5) }) }) \ No newline at end of file +val sss = Array(5) { Array>(5) { arrayOfNulls(5) } } \ No newline at end of file diff --git a/j2k/testData/fileOrElement/function/java8Lambdas.kt b/j2k/testData/fileOrElement/function/java8Lambdas.kt index 86c2392110c..229fc15c45c 100644 --- a/j2k/testData/fileOrElement/function/java8Lambdas.kt +++ b/j2k/testData/fileOrElement/function/java8Lambdas.kt @@ -18,29 +18,29 @@ public class Java8Class { } public fun foo() { - foo0({ "42" }) - foo0({ "42" }) - foo0({ + foo0 { "42" } + foo0 { "42" } + foo0 { helper() "42" - }) + } - foo1({ i -> "42" }) - foo1({ i -> "42" }) - foo1({ i: Int -> + foo1 { i -> "42" } + foo1 { i -> "42" } + foo1 { i: Int -> helper() if (i > 1) { return@foo1 "42" } "43" - }) + } - foo2({ i, j -> "42" }) - foo2({ i: Int, j: Int -> + foo2 { i, j -> "42" } + foo2 { i: Int, j: Int -> helper() "42" - }) + } val f = { i: Int, k: Int -> helper() @@ -68,19 +68,19 @@ public class Java8Class { val runnable = { } - foo1({ i: Int -> + foo1 { i: Int -> if (i > 1) { return@foo1 "42" } - foo0({ + foo0 { if (true) { return@foo0 "42" } "43" - }) + } "43" - }) + } } } \ No newline at end of file