-
Notifications
You must be signed in to change notification settings - Fork 63
Re-export 'ndk' and 'ndk_sys' crates #194
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
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.
Neat. I should really start looking into rust-mobile/ndk#478 so that we can use and at the same time also return those internal ndk
types back to users wherever needed/useful.
// Since we expose `ndk` types in our public API it's convenient if crates can | ||
// defer to these re-exported APIs and avoid having to bump explicit | ||
// dependencies when they pull in new releases of android-activity. |
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.
Reflow for line length?
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 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.
Right, it can't reflow dependencies
even though crates can
on the first line looks out of place.
I tried helix
which reflows it to 128
columns for some reason... Either way, IIRC they recently switched to an algorithm that tries to keep a balanced length while maintaining a maximum, rather than breaking whenever the next word would have gone over the limit.
825afb5
to
4b11d64
Compare
0251e7c
to
12223db
Compare
4b11d64
to
b570174
Compare
Since we expose `ndk` types in the public API it makes sense to re-export these APIs so users of android-activity can defer to these without needing to manually sync the versions for explicit dependencies.
12223db
to
602ccf8
Compare
Since we expose
ndk
types in the public API it makes sense to re-export these APIs so users of android-activity can defer to these without needing to manually sync the versions for explicit dependencies.