Skip to content
This repository has been archived by the owner on Apr 29, 2024. It is now read-only.

Add jni #9

Closed
wants to merge 23 commits into from
Closed

Add jni #9

wants to merge 23 commits into from

Conversation

mahesh-hegde
Copy link
Contributor

  • Adds JNI to FFI plugin, along with android support

  • There are several generated files which can be hard to review.

  • To verify reproducibility

    • run tool/gen_aux_methods.dart in jni/ folder

    • run dart run ffigen --config ffigen.yaml to generate bindings to jni.h ; The header file is copied from Android NDK plus some annotations for param names in function pointer fields. The ffigen currently links to my fork of ffigen with a jni-specific patch, on which changes from upstream ffigen should be easy to verify. I think that's better than committing all vendored files in the same commit.

Marking it as draft to get the CI right, eventually.

@coveralls
Copy link

coveralls commented Jul 7, 2022

Pull Request Test Coverage Report for Build 2635450534

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 2623143830: 0.0%
Covered Lines: 1
Relevant Lines: 1

💛 - Coveralls

@mahesh-hegde
Copy link
Contributor Author

mahesh-hegde commented Jul 8, 2022

Closing this PR in favor of new branch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants