Skip to content

Fix mismatched_lifetime_syntaxes warnings#50

Merged
henrahmagix merged 1 commit intogregjoy1:mainfrom
henrahmagix:cargo-fix
Nov 4, 2025
Merged

Fix mismatched_lifetime_syntaxes warnings#50
henrahmagix merged 1 commit intogregjoy1:mainfrom
henrahmagix:cargo-fix

Conversation

@henrahmagix
Copy link
Collaborator

Fixed automatically with cargo fix --lib -p redical_ical.

Warnings were:

warning: hiding a lifetime that's elided elsewhere is confusing
  --> redical_ical/src/lib.rs:44:17
   |
44 |     pub fn span(&self) -> &ParserInput {
   |                 ^^^^^     ^^^^^^^^^^^^
   |                 |         ||
   |                 |         |the same lifetime is hidden here
   |                 |         the same lifetime is elided here
   |                 the lifetime is elided here
   |
   = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
   = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
help: use `'_` for type paths
   |
44 |     pub fn span(&self) -> &ParserInput<'_> {
   |                                       ++++
warning: hiding a lifetime that's elided elsewhere is confusing
   --> redical_ical/src/lib.rs:176:33
    |
176 |     fn wrapped_grapheme_indices(&self, is_extended: bool) -> unicode_segmentation::GraphemeIndices;
    |                                 ^^^^^                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
    |                                 |
    |                                 the lifetime is elided here
    |
    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
help: use `'_` for type paths
    |
176 |     fn wrapped_grapheme_indices(&self, is_extended: bool) -> unicode_segmentation::GraphemeIndices<'_>;
    |                                                                                                   ++++
warning: hiding a lifetime that's elided elsewhere is confusing
   --> redical_ical/src/lib.rs:181:33
    |
181 |     fn wrapped_grapheme_indices(&self, is_extended: bool) -> unicode_segmentation::GraphemeIndices {
    |                                 ^^^^^                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
    |                                 |
    |                                 the lifetime is elided here
    |
    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
help: use `'_` for type paths
    |
181 |     fn wrapped_grapheme_indices(&self, is_extended: bool) -> unicode_segmentation::GraphemeIndices<'_> {
    |                                                                                                   ++++
warning: hiding a lifetime that's elided elsewhere is confusing
   --> redical_ical/src/lib.rs:188:33
    |
188 |     fn wrapped_grapheme_indices(&self, is_extended: bool) -> unicode_segmentation::GraphemeIndices {
    |                                 ^^^^^                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
    |                                 |
    |                                 the lifetime is elided here
    |
    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
help: use `'_` for type paths
    |
188 |     fn wrapped_grapheme_indices(&self, is_extended: bool) -> unicode_segmentation::GraphemeIndices<'_> {
    |

Fixed automatically with `cargo fix --lib -p redical_ical`
Copy link
Owner

@gregjoy1 gregjoy1 left a comment

Choose a reason for hiding this comment

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

Yes @henrahmagix 🌚 Good to go!

@henrahmagix henrahmagix merged commit d73ab8d into gregjoy1:main Nov 4, 2025
4 checks passed
@henrahmagix henrahmagix deleted the cargo-fix branch November 4, 2025 14:01
@henrahmagix
Copy link
Collaborator Author

ta! released as 0.2.6 =) https://github.com/gregjoy1/redical/releases/tag/0.2.6

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.

3 participants