Skip to content

[jnienv-gen] Generate a header file for the native functions #809

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 26, 2021

Conversation

grendello
Copy link
Contributor

In addition to generating a .c file, generate a .h one with declarations
of all the functions defined in the .c file. The header will be used by
the NET6 build of Xamarin.Android's native runtime.

Additionally, make it possible to not export symbols by defining the
JI_NO_VISIBILITY preprocessor macro. This will also be used by the
NET6 build of Xamarin.Android native runtime (which will no longer need
to make the symbols visible for p/invoke to work)

@jonpryor
Copy link
Contributor

Current failure in Java.Interop (Mac - .NET Core) is because mono crashed.

Restarted the tests.

@grendello grendello force-pushed the gen-ji-native-header branch from 365b7e9 to 260e61b Compare February 25, 2021 10:01
In addition to generating a .c file, generate a .h one with declarations
of all the functions defined in the .c file.  The header will be used by
the NET6 build of Xamarin.Android's native runtime.

Additionally, make it possible to not export symbols by defining the
`JI_NO_VISIBILITY` preprocessor macro.  This will also be used by the
NET6 build of Xamarin.Android native runtime (which will no longer need
to make the symbols visible for `p/invoke` to work)
@grendello grendello force-pushed the gen-ji-native-header branch from 260e61b to 82d1675 Compare February 25, 2021 10:35
@jonpryor jonpryor merged commit cd4c8f8 into dotnet:main Feb 26, 2021
@grendello grendello deleted the gen-ji-native-header branch February 26, 2021 08:16
@jpobst jpobst added this to the 11.3 (16.10 / 8.10) milestone Mar 1, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants