diff --git a/idea/src/org/jetbrains/jet/plugin/refactoring/changeSignature/JetChangeInfo.java b/idea/src/org/jetbrains/jet/plugin/refactoring/changeSignature/JetChangeInfo.java index 631109afa16..c52998a9958 100644 --- a/idea/src/org/jetbrains/jet/plugin/refactoring/changeSignature/JetChangeInfo.java +++ b/idea/src/org/jetbrains/jet/plugin/refactoring/changeSignature/JetChangeInfo.java @@ -125,7 +125,6 @@ public class JetChangeInfo implements ChangeInfo { JetParameterInfo parameterInfo = newParameters.get(i); if (i > 0) { buffer.append(","); - buffer.append("\n"); buffer.append(indent); } diff --git a/idea/testData/quickfix/changeSignature/afterAddConstructorParameter.kt b/idea/testData/quickfix/changeSignature/afterAddConstructorParameter.kt index b37f87989d8..46e025b3162 100644 --- a/idea/testData/quickfix/changeSignature/afterAddConstructorParameter.kt +++ b/idea/testData/quickfix/changeSignature/afterAddConstructorParameter.kt @@ -1,8 +1,7 @@ // "Add parameter to constructor 'Base'" "true" // DISABLE-ERRORS -open class Base(var x: Int, - d: Double) { +open class Base(var x: Int, d: Double) { val y = Base(1, 2.5); fun f() { diff --git a/idea/testData/quickfix/changeSignature/afterAddFunctionParameter.kt b/idea/testData/quickfix/changeSignature/afterAddFunctionParameter.kt index a359dab3cf0..ea126c90485 100644 --- a/idea/testData/quickfix/changeSignature/afterAddFunctionParameter.kt +++ b/idea/testData/quickfix/changeSignature/afterAddFunctionParameter.kt @@ -1,8 +1,7 @@ // "Add parameter to function 'foo'" "true" // DISABLE-ERRORS -fun foo(x: Int, - i: Int) { +fun foo(x: Int, i: Int) { foo(, 4); foo(1, 4); foo(1, 4); diff --git a/idea/testData/quickfix/changeSignature/afterAddFunctionParameterAndChangeTypes.kt b/idea/testData/quickfix/changeSignature/afterAddFunctionParameterAndChangeTypes.kt index 7709e69eb10..14629bbb076 100644 --- a/idea/testData/quickfix/changeSignature/afterAddFunctionParameterAndChangeTypes.kt +++ b/idea/testData/quickfix/changeSignature/afterAddFunctionParameterAndChangeTypes.kt @@ -1,10 +1,7 @@ // "Change the signature of function 'foo'" "true" // DISABLE-ERRORS -fun foo(x: Double, - i: Int, - i1: Int, - i2: Int) { +fun foo(x: Double, i: Int, i1: Int, i2: Int) { foo(,, 5, 6); foo(1,, 5, 6); foo(1, 2.5, 5, 6); diff --git a/idea/testData/quickfix/changeSignature/afterAddFunctionParameterLongNameRuntime.kt b/idea/testData/quickfix/changeSignature/afterAddFunctionParameterLongNameRuntime.kt index 1e20ae5a083..1a06247fcad 100644 --- a/idea/testData/quickfix/changeSignature/afterAddFunctionParameterLongNameRuntime.kt +++ b/idea/testData/quickfix/changeSignature/afterAddFunctionParameterLongNameRuntime.kt @@ -3,8 +3,7 @@ import kotlin.modules.ModuleBuilder -fun foo(x: Int, - moduleBuilder: ModuleBuilder) { +fun foo(x: Int, moduleBuilder: ModuleBuilder) { foo(, ModuleBuilder("", "")); foo(1, ModuleBuilder("", "")); foo(2, ModuleBuilder("", "")); diff --git a/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters1.kt b/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters1.kt index 1723a00db1a..190a2110f31 100644 --- a/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters1.kt +++ b/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters1.kt @@ -2,5 +2,5 @@ // DISABLE-ERRORS fun f(x: Int, y: Int, z : () -> Int) { - f(1, 2, {() -> x}); + f(1, 2, { x}); } diff --git a/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters2.kt b/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters2.kt index 325cedab369..cf06b743b43 100644 --- a/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters2.kt +++ b/idea/testData/quickfix/changeSignature/afterChangeFunctionLiteralParameters2.kt @@ -2,7 +2,5 @@ // DISABLE-ERRORS fun f(x: Int, y: Int, z : (Int, Int?, Any) -> Int) { - f(1, 2, {(i: Int, - i1: Int?, - any: Any) -> x}); + f(1, 2, {(i: Int, i1: Int?, any: Any) -> x}); } diff --git a/idea/testData/quickfix/changeSignature/afterComplexHierarchy.kt b/idea/testData/quickfix/changeSignature/afterComplexHierarchy.kt index 98d0f98547b..dcb4d1e310f 100644 --- a/idea/testData/quickfix/changeSignature/afterComplexHierarchy.kt +++ b/idea/testData/quickfix/changeSignature/afterComplexHierarchy.kt @@ -1,39 +1,32 @@ // "Add parameter to function 'f'" "true" trait OA { - fun f(a: Int, - i: Int) + fun f(a: Int, i: Int) } trait OB { - fun f(a: Int, - i: Int) + fun f(a: Int, i: Int) } trait O : OA, OB { - override fun f(a: Int, - i: Int) + override fun f(a: Int, i: Int) } trait OO : O { - override fun f(a: Int, - i: Int) { + override fun f(a: Int, i: Int) { } } trait OOO : OO { - override fun f(a: Int, - i: Int) {} + override fun f(a: Int, i: Int) {} } trait OOOA : OOO { - override fun f(a: Int, - i: Int) { + override fun f(a: Int, i: Int) { } } trait OOOB : OOO { - override fun f(a: Int, - i: Int) { + override fun f(a: Int, i: Int) { } } diff --git a/idea/testData/quickfix/changeSignature/afterComplexHierarchyTail.kt b/idea/testData/quickfix/changeSignature/afterComplexHierarchyTail.kt index adc9d53c20c..046fb3501ef 100644 --- a/idea/testData/quickfix/changeSignature/afterComplexHierarchyTail.kt +++ b/idea/testData/quickfix/changeSignature/afterComplexHierarchyTail.kt @@ -1,39 +1,32 @@ // "Add parameter to function 'f'" "true" trait OA { - fun f(a: Int, - s: String) + fun f(a: Int, s: String) } trait OB { - fun f(a: Int, - s: String) + fun f(a: Int, s: String) } trait O : OA, OB { - override fun f(a: Int, - s: String) + override fun f(a: Int, s: String) } trait OO : O { - override fun f(a: Int, - s: String) { + override fun f(a: Int, s: String) { } } trait OOO : OO { - override fun f(a: Int, - s: String) {} + override fun f(a: Int, s: String) {} } trait OOOA : OOO { - override fun f(a: Int, - s: String) { + override fun f(a: Int, s: String) { } } trait OOOB : OOO { - override fun f(a: Int, - s: String) { + override fun f(a: Int, s: String) { } } diff --git a/idea/testData/quickfix/changeSignature/afterLinearHierarchy.kt b/idea/testData/quickfix/changeSignature/afterLinearHierarchy.kt index d4388e53e1f..f7f4f8c093b 100644 --- a/idea/testData/quickfix/changeSignature/afterLinearHierarchy.kt +++ b/idea/testData/quickfix/changeSignature/afterLinearHierarchy.kt @@ -1,18 +1,15 @@ // "Add parameter to function 'f'" "true" trait O { - fun f(a: Int, - i: Int) + fun f(a: Int, i: Int) } trait OO : O { - override fun f(a: Int, - i: Int) { + override fun f(a: Int, i: Int) { } } trait OOO : OO { - override fun f(a: Int, - i: Int) {} + override fun f(a: Int, i: Int) {} } fun usage(o: O) { diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFun.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFun.kt index a3a86567326..9d481490390 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFun.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFun.kt @@ -1,6 +1,5 @@ // "Create parameter 'foo'" "true" -fun test(n: Int, - foo: Int) { +fun test(n: Int, foo: Int) { val t: Int = foo } \ No newline at end of file diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInClass.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInClass.kt index f53817ce462..c656d81f9eb 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInClass.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInClass.kt @@ -1,8 +1,7 @@ // "Create parameter 'foo'" "true" class A { - fun test(n: Int, - foo: Int) { + fun test(n: Int, foo: Int) { val t: Int = foo } } diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInGenClass.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInGenClass.kt index 69793ff6134..388c3b3e945 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInGenClass.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInFunInGenClass.kt @@ -1,8 +1,7 @@ // "Create parameter 'foo'" "true" class A { - fun test(n: Int, - foo: T) { + fun test(n: Int, foo: T) { val t: T = foo } } diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInClass.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInClass.kt index 2419b5251cc..a42704bc485 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInClass.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInClass.kt @@ -1,8 +1,7 @@ // "Create parameter 'foo'" "true" class A { - fun test(n: Int, - foo: T) { + fun test(n: Int, foo: T) { val t: T = foo } } diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInGenClass.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInGenClass.kt index bb2fa117eff..3c6b6e394f6 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInGenClass.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInGenFunInGenClass.kt @@ -1,8 +1,7 @@ // "Create parameter 'foo'" "true" class A { - fun test(n: Int, - foo: T) { + fun test(n: Int, foo: T) { val t: T = foo } } diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaNoParams.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaNoParams.kt index 3f3cb8fabe5..b2b757cc02e 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaNoParams.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaNoParams.kt @@ -1,7 +1,6 @@ // "Create parameter 'foo'" "true" // ACTION: Create local variable 'foo' -fun test(n: Int, - foo: Int) { +fun test(n: Int, foo: Int) { val f: () -> Int = { foo } } \ No newline at end of file diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithIt.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithIt.kt index 3fbc98f2fb7..2361d71a0fe 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithIt.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithIt.kt @@ -1,7 +1,6 @@ // "Create parameter 'foo'" "true" // ACTION: Create local variable 'foo' -fun test(n: Int, - foo: Int) { +fun test(n: Int, foo: Int) { val f: (Int) -> Int = { foo } } \ No newline at end of file diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithParams.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithParams.kt index c80574ecf78..b0d96fa8391 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithParams.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithParams.kt @@ -1,7 +1,6 @@ // "Create parameter 'foo'" "true" // ACTION: Create local variable 'foo' -fun test(n: Int, - foo: Int) { +fun test(n: Int, foo: Int) { val f: (Int, Int) -> Int = { (a, b) -> foo } } \ No newline at end of file diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithTypedParams.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithTypedParams.kt index 841673caf11..c78a7acf9ab 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithTypedParams.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInLambdaWithTypedParams.kt @@ -1,7 +1,6 @@ // "Create parameter 'foo'" "true" // ACTION: Create local variable 'foo' -fun test(n: Int, - foo: Any) { +fun test(n: Int, foo: Any) { val f = { (a: Int, b: Int) -> foo } } \ No newline at end of file diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInMultiLineLambdaNoParams.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInMultiLineLambdaNoParams.kt index 5998f4aaf69..1939b073197 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInMultiLineLambdaNoParams.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterInMultiLineLambdaNoParams.kt @@ -1,8 +1,7 @@ // "Create parameter 'foo'" "true" // ACTION: Create local variable 'foo' -fun test(n: Int, - foo: Int) { +fun test(n: Int, foo: Int) { val f: () -> Int = { foo } diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterNullableType.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterNullableType.kt index 617b1e99cb0..425ff1560b1 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterNullableType.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterNullableType.kt @@ -1,6 +1,5 @@ // "Create parameter 'foo'" "true" -fun test(n: Int, - foo: Int?) { +fun test(n: Int, foo: Int?) { val t: Int? = foo } \ No newline at end of file diff --git a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterWithPackageName.kt b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterWithPackageName.kt index ae6e3b9276b..5792eec76b2 100644 --- a/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterWithPackageName.kt +++ b/idea/testData/quickfix/createFromUsage/createVariable/parameter/afterWithPackageName.kt @@ -2,7 +2,6 @@ package foo -fun test(n: Int, - foo: Int) { +fun test(n: Int, foo: Int) { val t: Int = foo } \ No newline at end of file diff --git a/idea/testData/refactoring/changeSignature/ConstructorJavaUsagesAfter.kt b/idea/testData/refactoring/changeSignature/ConstructorJavaUsagesAfter.kt index 52a36d8ae8b..e98707e6b1e 100644 --- a/idea/testData/refactoring/changeSignature/ConstructorJavaUsagesAfter.kt +++ b/idea/testData/refactoring/changeSignature/ConstructorJavaUsagesAfter.kt @@ -1,4 +1,3 @@ -open class A(n: Int, - s: String) +open class A(n: Int, s: String) class B: A(1, "abc") \ No newline at end of file diff --git a/idea/testData/refactoring/changeSignature/ConstructorSwapArgumentsAfter.kt b/idea/testData/refactoring/changeSignature/ConstructorSwapArgumentsAfter.kt index b40132f6d0f..7dedc491d7a 100644 --- a/idea/testData/refactoring/changeSignature/ConstructorSwapArgumentsAfter.kt +++ b/idea/testData/refactoring/changeSignature/ConstructorSwapArgumentsAfter.kt @@ -1,6 +1,4 @@ -open class C1 protected (x3: ((Int) -> Int)?, - var _x2: Float, - val _x1: Int = 1) { +open class C1 protected (x3: ((Int) -> Int)?, var _x2: Float, val _x1: Int = 1) { fun bar() { val y1 = _x1; val y2 = _x2; diff --git a/idea/testData/refactoring/changeSignature/ExpressionFunctionAfter.kt b/idea/testData/refactoring/changeSignature/ExpressionFunctionAfter.kt index 6891478720a..eca46c606b0 100644 --- a/idea/testData/refactoring/changeSignature/ExpressionFunctionAfter.kt +++ b/idea/testData/refactoring/changeSignature/ExpressionFunctionAfter.kt @@ -1,2 +1 @@ -fun fun1(x1: Int, - y1: Int): Int = x1 * 2 + fun1(x1, ) +fun fun1(x1: Int, y1: Int): Int = x1 * 2 + fun1(x1, ) diff --git a/idea/testData/refactoring/changeSignature/FunctionJavaUsagesAndOverridesAfter.kt b/idea/testData/refactoring/changeSignature/FunctionJavaUsagesAndOverridesAfter.kt index 1cfdc090d29..6e7862604e8 100644 --- a/idea/testData/refactoring/changeSignature/FunctionJavaUsagesAndOverridesAfter.kt +++ b/idea/testData/refactoring/changeSignature/FunctionJavaUsagesAndOverridesAfter.kt @@ -1,9 +1,7 @@ open class A { - open fun foo(n: Int, - s: String): String = "" + open fun foo(n: Int, s: String): String = "" } class B: A() { - override fun foo(n: Int, - s: String): String = "" + override fun foo(n: Int, s: String): String = "" } \ No newline at end of file diff --git a/idea/testData/refactoring/changeSignature/FunctionLiteralAfter.kt b/idea/testData/refactoring/changeSignature/FunctionLiteralAfter.kt index 376f9e6f3f1..c8fac6d9292 100644 --- a/idea/testData/refactoring/changeSignature/FunctionLiteralAfter.kt +++ b/idea/testData/refactoring/changeSignature/FunctionLiteralAfter.kt @@ -1,5 +1,3 @@ fun foo() { - val v1 = {(z: Int, - y1: String, - x: Any): Int -> println(z); println(y1) } + val v1 = {(z: Int, y1: String, x: Any): Int -> println(z); println(y1) } } \ No newline at end of file diff --git a/idea/testData/refactoring/changeSignature/FunctionsAddRemoveArgumentsAfter.kt b/idea/testData/refactoring/changeSignature/FunctionsAddRemoveArgumentsAfter.kt index bb5ea602529..85f76ca5161 100644 --- a/idea/testData/refactoring/changeSignature/FunctionsAddRemoveArgumentsAfter.kt +++ b/idea/testData/refactoring/changeSignature/FunctionsAddRemoveArgumentsAfter.kt @@ -1,6 +1,4 @@ -fun foo(x0: Any?, - x1: Int = 1, - x2: Float) { +fun foo(x0: Any?, x1: Int = 1, x2: Float) { foo(null, 2, 3.5); val y1 = x1; val y2 = x2; diff --git a/idea/testData/refactoring/changeSignature/NoDefaultValuesInOverridesAfter.kt b/idea/testData/refactoring/changeSignature/NoDefaultValuesInOverridesAfter.kt index 8cd2f78a6a6..45c60318503 100644 --- a/idea/testData/refactoring/changeSignature/NoDefaultValuesInOverridesAfter.kt +++ b/idea/testData/refactoring/changeSignature/NoDefaultValuesInOverridesAfter.kt @@ -1,18 +1,15 @@ trait T { - fun foo(a: Int = 1, - b: String = "2") + fun foo(a: Int = 1, b: String = "2") } open class A: T { - override fun foo(a: Int, - b: String) { + override fun foo(a: Int, b: String) { throw UnsupportedOperationException() } } class B: A() { - override fun foo(a: Int, - b: String) { + override fun foo(a: Int, b: String) { throw UnsupportedOperationException() } } \ No newline at end of file diff --git a/idea/testData/refactoring/changeSignature/OverridesInEnumEntriesAfter.kt b/idea/testData/refactoring/changeSignature/OverridesInEnumEntriesAfter.kt index 8fccda67ee9..de37cc6936e 100644 --- a/idea/testData/refactoring/changeSignature/OverridesInEnumEntriesAfter.kt +++ b/idea/testData/refactoring/changeSignature/OverridesInEnumEntriesAfter.kt @@ -1,16 +1,13 @@ enum class E { - open fun foo(n: Int, - s: String): Int = n + open fun foo(n: Int, s: String): Int = n O A { - override fun foo(n: Int, - s: String): Int = n + 1 + override fun foo(n: Int, s: String): Int = n + 1 } B { - override fun foo(n: Int, - s: String): Int = n + 2 + override fun foo(n: Int, s: String): Int = n + 2 } } \ No newline at end of file