Skip to content

Remove unnecessary use<...> clauses after Rust 2024 edition upgrade#55

Merged
sile merged 4 commits intomasterfrom
remove-unnecessary-use
Sep 24, 2025
Merged

Remove unnecessary use<...> clauses after Rust 2024 edition upgrade#55
sile merged 4 commits intomasterfrom
remove-unnecessary-use

Conversation

@sile
Copy link
Copy Markdown
Owner

@sile sile commented Sep 24, 2025

No description provided.

@sile sile requested a review from Copilot September 24, 2025 23:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the codebase following the Rust 2024 edition upgrade by removing unnecessary use<...> clauses and simplifying function signatures. The changes eliminate redundant lifetime parameters and formatting that are no longer required in the newer Rust edition.

  • Remove use<...> bounds from impl Iterator return types that are now inferred automatically
  • Simplify lifetime parameters by removing unnecessary explicit lifetimes
  • Consolidate multi-line function signatures into single lines for better readability

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/tree.rs Simplifies function signature formatting for common_prefixes_owned
src/map.rs Removes use<...> clauses from iterator methods and simplifies lifetime parameters in iter_prefix methods

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sile sile merged commit cbd4661 into master Sep 24, 2025
12 checks passed
@sile sile deleted the remove-unnecessary-use branch September 24, 2025 23: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