Make intermediate *Impl properties traits, not classes

Otherwise multiple inheritance of classes would be needed to make
implementations inherit correctly for mutable properties

core/runtime.jvm/src/kotlin/reflect/jvm/internal/KCallableImpl.kt

@@ -16,6 +16,4 @@
 
 package kotlin.reflect.jvm.internal
 
-abstract class KCallableImpl<out R>(
-        public override val name: String
-) : KCallable<R>
+trait KCallableImpl<out R> : KCallable<R>

core/runtime.jvm/src/kotlin/reflect/jvm/internal/KExtensionPropertyImpl.kt

@@ -19,12 +19,12 @@ package kotlin.reflect.jvm.internal
 import java.lang.reflect.Method
 
 open class KExtensionPropertyImpl<T, out R>(
-        name: String,
+        public override val name: String,
         protected val owner: KPackageImpl,
         protected val receiverClass: Class<T>
-) : KExtensionProperty<T, R>, KPropertyImpl<R>(name) {
+) : KExtensionProperty<T, R>, KPropertyImpl<R> {
     // TODO: extract, make lazy (weak?), use our descriptors knowledge, support Java fields
-    protected val getter: Method = owner.jClass.getMethod(getterName(name), receiverClass)
+    val getter: Method = owner.jClass.getMethod(getterName(name), receiverClass)
 
     override fun get(receiver: T): R {
         return getter(null, receiver) as R
@@ -35,8 +35,8 @@ class KMutableExtensionPropertyImpl<T, R>(
         name: String,
         owner: KPackageImpl,
         receiverClass: Class<T>
-) : KMutableExtensionProperty<T, R>, KExtensionPropertyImpl<T, R>(name, owner, receiverClass) {
-    private val setter = owner.jClass.getMethod(setterName(name), receiverClass, getter.getReturnType()!!)
+) : KMutableExtensionProperty<T, R>, KMutablePropertyImpl<R>, KExtensionPropertyImpl<T, R>(name, owner, receiverClass) {
+    val setter = owner.jClass.getMethod(setterName(name), receiverClass, getter.getReturnType()!!)
 
     override fun set(receiver: T, value: R) {
         setter.invoke(null, receiver, value)

core/runtime.jvm/src/kotlin/reflect/jvm/internal/KMemberPropertyImpl.kt

@@ -20,9 +20,9 @@ import java.lang.reflect.Field
 import java.lang.reflect.Method
 
 open class KMemberPropertyImpl<T, out R>(
-        name: String,
+        public override val name: String,
         protected val owner: KClassImpl<T>
-) : KMemberProperty<T, R>, KPropertyImpl<R>(name) {
+) : KMemberProperty<T, R>, KPropertyImpl<R> {
     val field: Field? =
             if (owner.origin == KClassOrigin.FOREIGN) {
                 owner.jClass.getField(name)
@@ -48,7 +48,7 @@ open class KMemberPropertyImpl<T, out R>(
 class KMutableMemberPropertyImpl<T, R>(
         name: String,
         owner: KClassImpl<T>
-) : KMutableMemberProperty<T, R>, KMemberPropertyImpl<T, R>(name, owner) {
+) : KMutableMemberProperty<T, R>, KMutablePropertyImpl<R>, KMemberPropertyImpl<T, R>(name, owner) {
     val setter: Method? =
             if (owner.origin == KClassOrigin.KOTLIN) {
                 owner.jClass.getMaybeDeclaredMethod(setterName(name), getter!!.getReturnType()!!)

core/runtime.jvm/src/kotlin/reflect/jvm/internal/KPropertyImpl.kt

@@ -16,11 +16,12 @@
 
 package kotlin.reflect.jvm.internal
 
-abstract class KPropertyImpl<out R>(
-        name: String
-) : KProperty<R>, KCallableImpl<R>(name)
+import java.lang.reflect.*
+
+trait KPropertyImpl<out R> : KProperty<R>, KCallableImpl<R> {
+
+}
 
 
-abstract class KMutablePropertyImpl<R>(
-        name: String
-) : KMutableProperty<R>, KPropertyImpl<R>(name)
+trait KMutablePropertyImpl<R> : KMutableProperty<R>, KPropertyImpl<R> {
+}

core/runtime.jvm/src/kotlin/reflect/jvm/internal/KTopLevelPropertyImpl.kt

@@ -19,11 +19,11 @@ package kotlin.reflect.jvm.internal
 import java.lang.reflect.Method
 
 open class KTopLevelPropertyImpl<out R>(
-        name: String,
+        public override val name: String,
         protected val owner: KPackageImpl
-) : KTopLevelProperty<R>, KVariableImpl<R>(name) {
+) : KTopLevelProperty<R>, KVariableImpl<R> {
     // TODO: extract, make lazy (weak?), use our descriptors knowledge, support Java fields
-    protected val getter: Method = owner.jClass.getMethod(getterName(name))
+    val getter: Method = owner.jClass.getMethod(getterName(name))
 
     override fun get(): R {
         return getter(null) as R
@@ -33,8 +33,8 @@ open class KTopLevelPropertyImpl<out R>(
 class KMutableTopLevelPropertyImpl<R>(
         name: String,
         owner: KPackageImpl
-) : KMutableTopLevelProperty<R>, KTopLevelPropertyImpl<R>(name, owner) {
-    private val setter = owner.jClass.getMethod(setterName(name), getter.getReturnType()!!)
+) : KMutableTopLevelProperty<R>, KMutableVariableImpl<R>, KTopLevelPropertyImpl<R>(name, owner) {
+    val setter = owner.jClass.getMethod(setterName(name), getter.getReturnType()!!)
 
     override fun set(value: R) {
         setter.invoke(null, value)

core/runtime.jvm/src/kotlin/reflect/jvm/internal/KVariableImpl.kt

@@ -16,11 +16,6 @@
 
 package kotlin.reflect.jvm.internal
 
-abstract class KVariableImpl<out R>(
-        name: String
-) : KVariable<R>, KPropertyImpl<R>(name)
+trait KVariableImpl<out R> : KVariable<R>, KPropertyImpl<R>
 
-
-abstract class KMutableVariableImpl<R>(
-        name: String
-) : KMutableVariable<R>, KVariableImpl<R>(name)
+trait KMutableVariableImpl<R> : KMutableVariable<R>, KVariableImpl<R>, KMutablePropertyImpl<R>