Skip to content

Fix hidden lifetime warning - #201

Closed
rib wants to merge 1 commit into
mainfrom
rib/pr/fix-hidden-lifetime-warning
Closed

Fix hidden lifetime warning#201
rib wants to merge 1 commit into
mainfrom
rib/pr/fix-hidden-lifetime-warning

Conversation

@rib

@rib rib commented Aug 11, 2025

Copy link
Copy Markdown
Member

This fixes the current CI issue when building with the latest stable toolchain

/// This must only be called from your `android_main()` thread and it may panic if called
/// from another thread.
pub fn input_events_iter(&self) -> Result<input::InputIterator> {
pub fn input_events_iter(&'_ self) -> Result<input::InputIterator<'_>> {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If you apply the suggestion on self it does this. If you apply the suggestion on InputIterator I think it only suggests this:

Suggested change
pub fn input_events_iter(&'_ self) -> Result<input::InputIterator<'_>> {
pub fn input_events_iter(&self) -> Result<input::InputIterator<'_>> {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See also #184 where I fixed exactly this.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ah, yup, will merge that instead, thanks.

@rib

rib commented Aug 11, 2025

Copy link
Copy Markdown
Member Author

Will merge #184 instead

@rib rib closed this Aug 11, 2025
@MarijnS95
MarijnS95 deleted the rib/pr/fix-hidden-lifetime-warning branch August 11, 2025 15:08
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