Fix matching bad owner matching condition

In a previous change I assumed owner was using dots for inner classes,
  which is not the case and messes up with other parts of the Mixin
  support, so instead we keep the dollar sign separator when creating
  the MemberReference

Author: RedNesto

src/main/kotlin/platform/mixin/handlers/injectionPoint/NewInsnInjectionPoint.kt

@@ -219,7 +219,7 @@ private class NewInsnSelector(
 }
 
 private fun classToMemberReference(value: String): MemberReference? {
-    val fqn = value.replace('/', '.').replace('$', '.')
+    val fqn = value.replace('/', '.')
     if (fqn.isNotEmpty() && !fqn.startsWith('.') && !fqn.endsWith('.') && !fqn.contains("..")) {
         if (StringUtil.isJavaIdentifier(fqn.replace('.', '_'))) {
             return MemberReference("<init>", owner = fqn)

src/main/kotlin/util/MemberReference.kt

@@ -64,7 +64,7 @@ data class MemberReference(
 
     private fun matchOwner(clazz: String): Boolean {
         assert(!clazz.contains('.'))
-        return this.owner == null || this.owner == clazz.replace('/', '.').replace('$', '.')
+        return this.owner == null || this.owner == clazz.replace('/', '.')
     }
 
     override fun matchField(owner: String, name: String, desc: String): Boolean {