Kotlin unsigned type metadata bug #984

Activity

1. jpobst

   added

   bugComponent does not function as intendedComponent does not function as intended

   generatorIssues binding a Java library (generator, class-parse, etc.)Issues binding a Java library (generator, class-parse, etc.)

   on May 12, 2022
2. jpobst

   added this to the 7.0.0 milestone on May 12, 2022
3. jpobst commented on Jul 7, 2022

   

   jpobst

   on Jul 7, 2022

   ContributorAuthor

   More actions

   Notes for future me:

   fun contains(element: UInt): Boolean
   operator fun <T> Array<out T>.contains(element: T): Boolean

   In these cases, the Kotlin signature is the same, thus the mangled Java method name is the same:

     public boolean contains-WZ4Q5Ns(int element) {
       return contains-WZ4Q5Ns(this.storage, element);
     }
     
     public static boolean contains-WZ4Q5Ns(int[] arg0, int element) {
       Intrinsics.checkNotNullParameter(arg0, "arg0");
       return ArraysKt.contains(arg0, element);
     }

   The mangled part seems to be a hash of the Kotlin parameter types of the method. Because they "match", Kotlin does not write its metadata for each method, it only writes one copy which is correct-ish for both methods:

   

   (Note that the JvmSignature is ([II)Z even though there is only one ValueParameter.

   Thus, when more than one Java method matches the JvmName and the JvmName is "mangled" (contains a hyphen), we should apply the parameter fixups to all matching methods, even though the JvmSignature doesn't match.
4. jonpryor

   closed this as completedin #1009on Jul 14, 2022
5. jonpryor

   added 2 commits that reference this issue on Jul 14, 2022

   [generator] Kotlin metadata can apply to multiple Java method (#1009)

   ...

   275fa75

   Bump to xamarin/java.interop/main@032f1e71 (#7180)

   ...

   ace5f71
6. github-actions

   locked and limited conversation to collaborators on Apr 12, 2024
7. pull

   added a commit that references this issue on Jul 1, 2026

   [generator] Kotlin metadata can apply to multiple Java method (#1009)

   ...

   3fb77a6