Skip to content

JavaCallableWrapperGenerator uses XML? #11921

Description

@jonpryor

Context: https://xamarinhq.slack.com/archives/C03CEGRUW/p1574437340441800

Java.Interop.Tools.JavaCallableWrappers.JavaCallableWrapperGenerator reads assemblies via Mono.Cecil in order to write Java Callable Wrappers.

Is this actually desirable?

One of the impacts to this decision is that the Xamarin.Android linker can't fully link; it must preserve [Register] custom attributes so that the Java Callable Wrappers can be generated, and then a later linker step must be run to remove all of the [Register] attributes:

i wonder if the linker could generate a file of "here's the remaining things that need java stubs (with Register data)", then[JavaCallableWrapperGenerator] could work from that file instead of needing to Cecil the assembly, and then linker could remove [Register] at the same time, so we don't need to do that Cecil either
cut out 2 full cecil loads and enumerations

There have also been mutterings of migrating parts of the <GenerateJavaStubs/> task to use System.Reflection.Metadata from Cecil, or to make it suitable for use with Incremental Builds.

It may be desirable to have JavaCallableWrapperGenerator instead process an XML description of the types to generate, possibly using the same XML format as generator's api.xml files, and then update the linker to generate this XML file as part of it's operation.

This would allow JavaCallableWrapperGenerator to not use Mono.Cecil in order to operate, which may be desirable.

Activity

  1. jpobst commented on Dec 14, 2019

    @jpobst
    Contributor

    Just some notes that were raised:

    • The linker (generally) only runs on Release builds, so this would not help Debug builds
    • We would still need to use Cecil (or something) for Debug builds to build the wrappers
  2. jonpryor commented on Dec 13, 2023

    @jonpryor
    ContributorAuthor

    More recent random thoughts…

    We're now ".NET", not Xamarin.Android, and as part of that migration we now use the .NET linker in the Release configuration.

    The .NET linker runs a pipeline of linker steps, which .NET Android controls via the @(_TrimmerCustomSteps) item group.

    A thought of ours, to "force" cleanup of the <GenerateJavaStubs/> task, is to move Java Callable Wrapper generation into a linker pipeline step. (This feels quite plausible and sane.)

    .NET Android Debug builds also need to do "linker-like" things, such as GC.KeepAlive() insertion for older binding assemblies and fixing "missing" abstract methods, so it occurs to us that Debug builds could also use a (shorter!) linker pipeline, which could also be used to emit Java Callable Wrappers. (The big unknown here is performance: how much slower would this be? Could we have an "in-process linker pipeline"?)

    If these ideas pan out, it means that this idea is moot: there wouldn't need to be an XML intermediary, because Java Callable Wrapper generation would be a linker step which could run before [Register] is removed, and then [Register] could be removed in a subsequent linker step in the same pipeline. Furthermore, the .NET linker already uses Cecil, so use of Cecil would not be a new concern.

  3. added theissue type on Apr 29, 2025
  4. removed
    enhancementProposed change to current functionality.
    on Apr 29, 2025
  5. added
    java-interopIssues migrated from dotnet/java-interop / relates to the Java.Interop subtree
    needs-triageIssues that need to be assigned.
    Area: BindingsIssues in Java Library Binding projects.
    on Jul 1, 2026
  6. dotnet-policy-service commented on Jul 1, 2026

    @dotnet-policy-service

    We suspect this issue is stale and no longer relevant. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will undo this process.

  7. dotnet-policy-service commented on Jul 15, 2026

    @dotnet-policy-service

    This issue will now be closed since it had been marked "possibly-stale" but received no further activity in the past 14 days. It is still possible to reopen or comment on the issue, but please note that the issue will be locked if it remains inactive for another 30 days.

  8. locked and limited conversation to collaborators on Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Area: BindingsIssues in Java Library Binding projects.java-interopIssues migrated from dotnet/java-interop / relates to the Java.Interop subtreeneeds-triageIssues that need to be assigned.possibly-staleIssues that are potentially no longer relevant.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions