8000 Re-export 'ndk' and 'ndk_sys' crates by rib · Pull Request #194 · rust-mobile/android-activity · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 1 commit into from
Apr 1, 2025
Merged

Re-export 'ndk' and 'ndk_sys' crates #194

merged 1 commit into from
Apr 1, 2025

Conversation

rib
Copy link
Member
@rib rib commented Apr 1, 2025

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.

@rib rib requested a review from MarijnS95 April 1, 2025 11:36
Copy link
Member
@MarijnS95 MarijnS95 left a 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.

Comment on lines +127 to +130
// 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.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reflow for line length?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'dependencies' is a surprisingly long word I suppose :)

I use a vscode plugin for wrapping at various column widths and this is already wrapped:
image

Copy link
Member
@MarijnS95 MarijnS95 Apr 1, 2025

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.

@rib rib force-pushed the rib/pr/bump-rust-version branch from 825afb5 to 4b11d64 Compare April 1, 2025 14:25
@rib rib force-pushed the rib/pr/ndk-re-export branch 2 times, most recently from 0251e7c to 12223db Compare April 1, 2025 14:26
@rib rib force-pushed the rib/pr/bump-rust-version branch from 4b11d64 to b570174 Compare April 1, 2025 14:26
Base automatically changed from rib/pr/bump-rust-version to main April 1, 2025 14:30
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.
@rib rib force-pushed the rib/pr/ndk-re-export branch from 12223db to 602ccf8 Compare April 1, 2025 14:35
@rib rib merged commit ac2e17e into main Apr 1, 2025
3 checks passed
@rib rib deleted the rib/pr/ndk-re-export branch April 1, 2025 14:38
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.

2 participants
0