-
Notifications
You must be signed in to change notification settings - Fork 715
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
doc: update Java interop document #5783
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are other mentions of the C bindings still available in the text. Please also change them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mentioned that there are other places where the C bindings are not removed.
However I cannot point you to them, as GitHub doesn't let me comment on unchanged lines.
It would be nice to allow changes, so we can commit to your fork.
So I'll try to review using links to the lines:
-To generate the Dart (and C) bindings, run `jnigen` and
+To generate the Dart bindings, run `jnigen` and
-you must build the dynamic libraries for `jni` and the generated C files.
+you must build the dynamic library for `jni`.
@HosseinYousefi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks @utamori!
@parlough Please merge whenever you see fit. |
/gcbrun |
Visit the preview URL for this PR (updated for commit 430acaf): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you both!
Pure Dart binding is now standard
This page needs to be updated so that newcomers are not confused
HosseinYousefi/jnigen_example#2
Contribution guidelines:
dart format
.<?code-excerpt
need to be updated in their source.dart
file as well.