New J2K: add missing line break between property and getter
#KT-35739 fixed
This commit is contained in:
+1
@@ -182,6 +182,7 @@ private class ConvertGettersAndSettersToPropertyStatefulProcessing(
|
|||||||
|
|
||||||
val ktGetter = factory.createGetter(body, getter.modifiersText)
|
val ktGetter = factory.createGetter(body, getter.modifiersText)
|
||||||
ktGetter.filterModifiers()
|
ktGetter.filterModifiers()
|
||||||
|
property.add(factory.createNewLine(1))
|
||||||
return property.add(ktGetter).cast<KtPropertyAccessor>().also {
|
return property.add(ktGetter).cast<KtPropertyAccessor>().also {
|
||||||
if (getter is RealGetter) {
|
if (getter is RealGetter) {
|
||||||
getter.function.forAllUsages { usage ->
|
getter.function.forAllUsages { usage ->
|
||||||
|
|||||||
+2
-1
@@ -4,5 +4,6 @@ interface Parent {
|
|||||||
} //Subclass
|
} //Subclass
|
||||||
|
|
||||||
class Child : Parent {
|
class Child : Parent {
|
||||||
override val x: Unit get() {}
|
override val x: Unit
|
||||||
|
get() {}
|
||||||
}
|
}
|
||||||
@@ -0,0 +1,10 @@
|
|||||||
|
public class Foo {
|
||||||
|
private String value;
|
||||||
|
|
||||||
|
private String getValue() {
|
||||||
|
if (true) {
|
||||||
|
value = "new";
|
||||||
|
}
|
||||||
|
return value;
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,9 @@
|
|||||||
|
class Foo {
|
||||||
|
private var value: String? = null
|
||||||
|
private get() {
|
||||||
|
if (true) {
|
||||||
|
field = "new"
|
||||||
|
}
|
||||||
|
return field
|
||||||
|
}
|
||||||
|
}
|
||||||
+3
-1
@@ -1,4 +1,6 @@
|
|||||||
class Foo {
|
class Foo {
|
||||||
private external fun nativeMethod()
|
private external fun nativeMethod()
|
||||||
var bar: Int external get external set
|
var bar: Int
|
||||||
|
external get
|
||||||
|
external set
|
||||||
}
|
}
|
||||||
+5
@@ -1595,6 +1595,11 @@ public class NewJavaToKotlinConverterSingleFileTestGenerated extends AbstractNew
|
|||||||
runTest("nj2k/testData/newJ2k/detectProperties/kt-32253.java");
|
runTest("nj2k/testData/newJ2k/detectProperties/kt-32253.java");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@TestMetadata("kt-35739.java")
|
||||||
|
public void testKt_35739() throws Exception {
|
||||||
|
runTest("nj2k/testData/newJ2k/detectProperties/kt-35739.java");
|
||||||
|
}
|
||||||
|
|
||||||
@TestMetadata("NoBackingField.java")
|
@TestMetadata("NoBackingField.java")
|
||||||
public void testNoBackingField() throws Exception {
|
public void testNoBackingField() throws Exception {
|
||||||
runTest("nj2k/testData/newJ2k/detectProperties/NoBackingField.java");
|
runTest("nj2k/testData/newJ2k/detectProperties/NoBackingField.java");
|
||||||
|
|||||||
Reference in New Issue
Block a user