Adjust various testData to size transformation

compiler/testData/codegen/box/builtinStubMethods/Collection.kt

@@ -1,5 +1,5 @@
 class MyCollection<T>: Collection<T> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<T> = throw UnsupportedOperationException()

compiler/testData/codegen/box/builtinStubMethods/List.kt

@@ -1,5 +1,5 @@
 class MyList<T>: List<T> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<T> = throw Error()

compiler/testData/codegen/box/builtinStubMethods/ListWithAllImplementations.kt

@@ -1,5 +1,5 @@
 class MyList<T>(val v: T): List<T> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<T> = throw Error()

compiler/testData/codegen/box/builtinStubMethods/ListWithAllInheritedImplementations.kt

@@ -12,7 +12,7 @@ open class Super<T>(val v: T) {
 }
 
 class MyList<T>(v: T): Super<T>(v), List<T> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<T> = throw Error()

compiler/testData/codegen/box/builtinStubMethods/Map.kt

@@ -1,5 +1,5 @@
 class MyMap<K, V>: Map<K, V> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun containsKey(key: Any?): Boolean = false
     override fun containsValue(value: Any?): Boolean = false

compiler/testData/codegen/box/builtinStubMethods/MapWithAllImplementations.kt

@@ -1,5 +1,5 @@
 class MyMap<K, V>: Map<K, V> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun containsKey(key: Any?): Boolean = false
     override fun containsValue(value: Any?): Boolean = false

compiler/testData/codegen/box/builtinStubMethods/SubstitutedList.kt

@@ -1,5 +1,5 @@
 class MyList: List<String> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<String> = throw Error()

compiler/testData/codegen/box/builtinStubMethods/extendJavaCollections/abstractList.kt

@@ -2,7 +2,7 @@ import java.util.AbstractList
 
 class A : AbstractList<String>() {
     override fun get(index: Int): String = ""
-    override fun size(): Int = 0
+    override val size: Int get() = 0
 }
 
 fun box(): String {

compiler/testData/codegen/box/builtinStubMethods/implementationInTrait.kt

@@ -3,7 +3,7 @@ interface Addable {
 }
 
 class C : Addable, List<String> {
-    override fun size(): Int = null!!
+    override val size: Int get() = null!!
     override fun isEmpty(): Boolean = null!!
     override fun contains(o: Any?): Boolean = null!!
     override fun iterator(): Iterator<String> = null!!

compiler/testData/codegen/box/builtinStubMethods/inheritedImplementations.kt

@@ -5,7 +5,7 @@ open class SetStringImpl {
 }
 
 class S : Set<String>, SetStringImpl() {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<String> = null!!

compiler/testData/codegen/box/builtinStubMethods/manyTypeParametersWithUpperBounds.kt

@@ -1,7 +1,7 @@
 import java.util.Collections
 
 class A<U : Number, V : U, W : V> : Set<W> {
-    override fun size(): Int = 0
+    override val size: Int get() = 0
     override fun isEmpty(): Boolean = true
     override fun contains(o: Any?): Boolean = false
     override fun iterator(): Iterator<W> = Collections.emptySet<W>().iterator()

compiler/testData/codegen/box/builtinStubMethods/nonTrivialSubstitution.kt

@@ -2,7 +2,7 @@ import java.util.ArrayList
 
 class MyCollection<T> : Collection<List<Iterator<T>>> {
     override fun iterator() = null!!
-    override fun size(): Int = null!!
+    override val size: Int get() = null!!
     override fun isEmpty(): Boolean = null!!
     override fun contains(o: Any?): Boolean = null!!
     override fun containsAll(c: Collection<Any?>): Boolean = null!!

compiler/testData/codegen/box/classes/initializerBlockDImpl.kt

@@ -15,6 +15,6 @@ class World() {
 
 fun box() : String {
   val w = World()
-  if (w.items.size() != 1) return "fail"
+  if (w.items.size != 1) return "fail"
   return "OK"
 }

compiler/testData/codegen/box/classes/kt2395.kt

@@ -3,7 +3,7 @@ import java.util.AbstractList
 class MyList(): AbstractList<String>() {
      public fun getModificationCount(): Int = modCount
      public override fun get(index: Int): String = ""
-     public override fun size(): Int = 0
+     public override val size: Int get() = 0
 }
 
 fun box(): String {

compiler/testData/codegen/box/classes/kt343.kt

@@ -20,5 +20,5 @@ fun box(): String {
     }
     bar()
 
-    return if (list.size() == 2 && list.get(0) == 2 && list.get(1) == 3) "OK" else "fail"
+    return if (list.size == 2 && list.get(0) == 2 && list.get(1) == 3) "OK" else "fail"
 }

compiler/testData/codegen/box/classes/overloadBinaryOperator.kt

@@ -21,5 +21,5 @@ fun box(): String {
     v1.add("foo")
     v2.add("bar")
     val v3 = v1 + v2
-    return if (v3.contents.size() == 2) "OK" else "fail"
+    return if (v3.contents.size == 2) "OK" else "fail"
 }

compiler/testData/codegen/box/classes/overloadPlusAssign.kt

@@ -22,5 +22,5 @@ fun box(): String {
     v1.add("foo")
     v2.add("bar")
     v1 += v2
-    return if (v1.contents.size() == 2) "OK" else "fail"
+    return if (v1.contents.size == 2) "OK" else "fail"
 }

compiler/testData/codegen/box/classes/overloadPlusAssignReturn.kt

@@ -26,5 +26,5 @@ fun box(): String {
     val v3 = v1
     v2.add("bar")
     v1 += v2
-    return if (v1.contents.size() == 2 && v3.contents.size() == 1) "OK" else "fail"
+    return if (v1.contents.size == 2 && v3.contents.size == 1) "OK" else "fail"
 }

compiler/testData/codegen/box/classes/overloadPlusToPlusAssign.kt

@@ -26,5 +26,5 @@ fun box(): String {
     val v3 = v1
     v2.add("bar")
     v1 += v2
-    return if (v1.contents.size() == 2 && v3.contents.size() == 1) "OK" else "fail"
+    return if (v1.contents.size == 2 && v3.contents.size == 1) "OK" else "fail"
 }

compiler/testData/codegen/box/delegatedProperty/propertyMetadataEqualsHashCodeToString.kt

@@ -25,7 +25,7 @@ fun box(): String {
     foo
     42.foo
 
-    if (O.metadatas.size() != 1)
+    if (O.metadatas.size != 1)
         return "Too many different PropertyMetadata instances: ${O.metadatas}"
 
     val m = O.metadatas.iterator().next()

compiler/testData/codegen/box/delegatedProperty/propertyMetadataShouldBeCached.kt

@@ -39,7 +39,7 @@ fun box(): String {
     A.bar = baz + a.foo
     baz + A.bar
 
-    if (metadatas.keySet().size() != 3)
+    if (metadatas.keySet().size != 3)
         return "Fail: only three instances of PropertyMetadata should have been created\n${metadatas.keySet()}"
 
     return "OK"

compiler/testData/codegen/box/extensionFunctions/kt475.kt

@@ -10,9 +10,9 @@ fun box() : String {
 }
 
 var <T> ArrayList<T>.length : Int
-    get() = size()
+    get() = size
     set(value: Int) = throw java.lang.Error()
 
 var <T> ArrayList<T>.last : T
-    get() = get(size()-1)!!
-    set(el : T) { set(size()-1, el) }
+    get() = get(size-1)!!
+    set(el : T) { set(size-1, el) }

compiler/testData/codegen/box/extensionFunctions/kt865.kt

@@ -15,5 +15,5 @@ class Template() {
 fun box() : String {
     val u = Template()
     u.test()
-    return if(u.collected.size() == 1 && u.collected.get(0) == "239") "OK" else "fail"
+    return if(u.collected.size == 1 && u.collected.get(0) == "239") "OK" else "fail"
 }

compiler/testData/codegen/box/vararg/kt581.kt

@@ -12,7 +12,7 @@ fun box() : String {
     for (i in vals.indices)
         for (j in i..vals.lastIndex())
              diffs.add(vals[i] - vals[j])
-    val size = diffs.size()
+    val size = diffs.size
 
     if (size != 8) return "Fail $size"
     return "OK"

compiler/testData/codegen/boxWithJava/builtinStubMethods/substitutedList/substitutedList.kt

@@ -1,5 +1,5 @@
 abstract class C : Test.A, List<String> {
-    override fun size(): Int = null!!
+    override val size: Int get() = null!!
     override fun isEmpty(): Boolean = null!!
     override fun contains(o: Any?): Boolean = null!!
     override fun iterator(): Iterator<String> = null!!

compiler/testData/codegen/controlStructures/ifBlock.kt

@@ -3,7 +3,7 @@ import java.util.*
 fun concat(l: List<String>): String? {
     val sb = StringBuilder()
     for(s in l) {
-        val x = if(l.size() > 1) { "T" } else { "F" };
+        val x = if(l.size > 1) { "T" } else { "F" };
         sb.append(x)
     }
     return sb.toString()

compiler/testData/diagnostics/tests/inference/regressions/kt2179.kt

@@ -43,4 +43,4 @@ fun <T, C: MutableCollection<in T>> Array<T>.toCollection(result: C) : C {
 }
 
 val Collection<*>.size : Int
-    get() = size()
+    get() = size

compiler/testData/diagnostics/tests/library/Collections.kt

@@ -1,7 +1,7 @@
 package collections
 
 fun <T> testCollection(c: Collection<T>, t: T) {
-    c.size()
+    c.size
     c.isEmpty()
     c.contains(1)
     val <!UNUSED_VARIABLE!>iterator<!>: Iterator<T> = c.iterator()
@@ -17,7 +17,7 @@ fun <T> testCollection(c: Collection<T>, t: T) {
 
 }
 fun <T> testMutableCollection(c: MutableCollection<T>, t: T) {
-    c.size()
+    c.size
     c.isEmpty()
     c.contains(1)
     val <!UNUSED_VARIABLE!>iterator<!>: Iterator<T> = c.iterator()
@@ -59,7 +59,7 @@ fun <T> testMutableList(l: MutableList<T>, t: T) {
 }
 
 fun <T> testSet(s: Set<T>, t: T) {
-    s.size()
+    s.size
     s.isEmpty()
     s.contains(1)
     val <!UNUSED_VARIABLE!>iterator<!>: Iterator<T> = s.iterator()
@@ -75,7 +75,7 @@ fun <T> testSet(s: Set<T>, t: T) {
 
 }
 fun <T> testMutableSet(s: MutableSet<T>, t: T) {
-    s.size()
+    s.size
     s.isEmpty()
     s.contains(1)
     val <!UNUSED_VARIABLE!>iterator<!>: Iterator<T> = s.iterator()

compiler/testData/diagnostics/tests/scopes/initializerScopeOfExtensionProperty.kt

@@ -1,8 +1,8 @@
 package i
 
-val <T> List<T>.length = <!EXTENSION_PROPERTY_WITH_BACKING_FIELD!>size()<!>
+val <T> List<T>.length = <!EXTENSION_PROPERTY_WITH_BACKING_FIELD!>size<!>
 
-val <T> List<T>.length1 : Int get() = size()
+val <T> List<T>.length1 : Int get() = size
 
 val String.bd = <!EXTENSION_PROPERTY_WITH_BACKING_FIELD!>this + "!"<!>
 

compiler/testData/diagnostics/tests/smartCasts/safecalls/anotherVal.kt

@@ -2,5 +2,5 @@ fun calc(x: List<String>?, y: Int?): Int {
     x?.get(y!! - 1) 
     // y!! above should not provide smart cast here
     val yy: Int = <!TYPE_MISMATCH!>y<!>
-    return  yy + (x?.size() ?: 0)
+    return  yy + (x?.size ?: 0)
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/argument.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?): Int {
     // After KT-5840 fix !! assertion should become unnecessary here
-    x?.get(x<!UNNECESSARY_NOT_NULL_ASSERTION!>!!<!>.size() - 1) 
+    x?.get(x<!UNNECESSARY_NOT_NULL_ASSERTION!>!!<!>.size - 1)
     // x?. or x!! above should not provide smart cast here
-    return x<!UNSAFE_CALL!>.<!>size()
+    return x<!UNSAFE_CALL!>.<!>size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/chainAndUse.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?): Int {
     // x should be non-null in arguments list, despite of a chain
-    x?.subList(0, 1)?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size())
+    x?.subList(0, 1)?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size)
     // But not here!
-    return x!!.size()
+    return x!!.size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/chainInChain.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?, y: List<Int>?) {
     // x and y should be non-null in arguments list, despite of a chains
-    x?.subList(y?.subList(1, 2)?.get(<!DEBUG_INFO_SMARTCAST!>y<!>.size()) ?: 0, 
+    x?.subList(y?.subList(1, 2)?.get(<!DEBUG_INFO_SMARTCAST!>y<!>.size) ?: 0,
                y?.get(0) ?: 1) // But safe call is NECESSARY here for y
-     ?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size()) 
+     ?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size)
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/chainMixedUnsafe.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?): Int {
     // x should be non-null in arguments list, despite of a chain
-    x?.subList(0, 1)<!UNSAFE_CALL!>.<!>get(<!DEBUG_INFO_SMARTCAST!>x<!>.size())
+    x?.subList(0, 1)<!UNSAFE_CALL!>.<!>get(<!DEBUG_INFO_SMARTCAST!>x<!>.size)
     // But not here!
-    return x!!.size()
+    return x!!.size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/doubleCall.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?): Int {
     // x should be non-null in arguments list, including inner call
-    x?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size() - 1).length()) 
+    x?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size - 1).length())
     // but not also here!
-    return x<!UNSAFE_CALL!>.<!>size()               
+    return x<!UNSAFE_CALL!>.<!>size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/falseArgument.kt

@@ -1,7 +1,7 @@
 fun foo(y: Int) = y
 
 fun calc(x: List<String>?): Int {
-    foo(x!!.size())
+    foo(x!!.size)
     // Here we should have smart cast because of x!!, despite of KT-7204 fixed
-    return <!DEBUG_INFO_SMARTCAST!>x<!>.size()
+    return <!DEBUG_INFO_SMARTCAST!>x<!>.size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/falseExtension.kt

@@ -1,7 +1,7 @@
 fun String.foo(y: Int) = y
 
 fun calc(x: List<String>?): Int {
-    "abc".foo(x!!.size())
+    "abc".foo(x!!.size)
     // Here we should have smart cast because of x!!, despite of KT-7204 fixed
-    return <!DEBUG_INFO_SMARTCAST!>x<!>.size()
+    return <!DEBUG_INFO_SMARTCAST!>x<!>.size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/insideCall.kt

@@ -4,5 +4,5 @@ fun foo(y: Int): Int {
 
 fun calc(x: List<String>?): Int {
     // x should be non-null in arguments list
-    return foo(x?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size() - 1)!!.length())
+    return foo(x?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size - 1)!!.length())
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/longChain.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?) {
     // x should be non-null in arguments list, despite of a chain
-    x?.subList(0, <!DEBUG_INFO_SMARTCAST!>x<!>.size())?.
-       subList(0, <!DEBUG_INFO_SMARTCAST!>x<!>.size())?.
-       get(<!DEBUG_INFO_SMARTCAST!>x<!>.size()) 
+    x?.subList(0, <!DEBUG_INFO_SMARTCAST!>x<!>.size)?.
+       subList(0, <!DEBUG_INFO_SMARTCAST!>x<!>.size)?.
+       get(<!DEBUG_INFO_SMARTCAST!>x<!>.size)
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/simple.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?): Int {
     // x should be non-null in arguments list
-    x?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size() - 1) 
+    x?.get(<!DEBUG_INFO_SMARTCAST!>x<!>.size - 1)
     // but not also here!
-    return x<!UNSAFE_CALL!>.<!>size()               
+    return x<!UNSAFE_CALL!>.<!>size
 }

compiler/testData/diagnostics/tests/smartCasts/safecalls/twoArgs.kt

@@ -1,6 +1,6 @@
 fun calc(x: List<String>?): Int {
     // x should be non-null in arguments list
-    x?.subList(<!DEBUG_INFO_SMARTCAST!>x<!>.size() - 1, <!DEBUG_INFO_SMARTCAST!>x<!>.size()) 
+    x?.subList(<!DEBUG_INFO_SMARTCAST!>x<!>.size - 1, <!DEBUG_INFO_SMARTCAST!>x<!>.size)
     // but not also here!
-    return x<!UNSAFE_CALL!>.<!>size()               
+    return x<!UNSAFE_CALL!>.<!>size
 }

compiler/testData/diagnostics/tests/substitutions/starProjections.kt

@@ -19,6 +19,6 @@ fun testB(b: B<*, *>) {
     b.r().checkType { _<Any?>() }
     b.t().checkType { _<B<List<*>, *>>() }
 
-    b.t().r().size()
+    b.t().r().size
 }