Skip to content

Split reflection-based JniTypeManager behavior#1441

Draft
simonrozsival wants to merge 2 commits into
mainfrom
dev/simonrozsival/fix-aot-warnings-jni-type-manager
Draft

Split reflection-based JniTypeManager behavior#1441
simonrozsival wants to merge 2 commits into
mainfrom
dev/simonrozsival/fix-aot-warnings-jni-type-manager

Conversation

@simonrozsival
Copy link
Copy Markdown
Member

Draft PR to let CI validate the JniTypeManager split and AOT warning work.

Summary:

  • Keep JniTypeManager as the abstract base and move reflection-backed behavior into DynamicJniTypeManager.
  • Update JRE/test managers that extend the default behavior to inherit from DynamicJniTypeManager.
  • Keep NativeAOT sample managers on the abstract base with explicit mappings instead of reflection-backed base calls.
  • Update public API baselines for the split.

Local validation:

  • dotnet build src/Java.Interop/Java.Interop.csproj -p:JavaCPath="$JAVA_HOME/bin/javac" -p:JarPath="$JAVA_HOME/bin/jar"
  • dotnet build tests/Java.Interop-Tests/Java.Interop-Tests.csproj -p:JavaCPath="$JAVA_HOME/bin/javac" -p:JarPath="$JAVA_HOME/bin/jar" -p:NativeToolchainSupported=false

Notes:

  • Full local prepare/build is still blocked in this workspace by local build setup/toolchain issues, so this is intentionally draft for CI coverage.

@simonrozsival simonrozsival force-pushed the dev/simonrozsival/fix-aot-warnings-jni-type-manager branch from 5ff4c17 to 6d72140 Compare June 4, 2026 14:24
@simonrozsival simonrozsival force-pushed the dev/simonrozsival/fix-aot-warnings-jni-type-manager branch from 6d72140 to 58f0dbd Compare June 4, 2026 14:30
@simonrozsival
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant