Skip to content

[Clippy] Fix clippy issues in libgit2-sys crate - #1299

Merged
weihanglo merged 3 commits into
rust-lang:mainfrom
DanielEScherzer:clippy-fixes-libgit2-sys
Jul 20, 2026
Merged

[Clippy] Fix clippy issues in libgit2-sys crate#1299
weihanglo merged 3 commits into
rust-lang:mainfrom
DanielEScherzer:clippy-fixes-libgit2-sys

Conversation

@DanielEScherzer

Copy link
Copy Markdown
Contributor

Commits are split by file and lint for ease of review.

None of the issues required keeping the old code with an #[expect] attribute, they could all be fixed.

@rustbot rustbot added the S-waiting-on-review Status: Waiting on review label Jul 19, 2026
…t2-sys/build.rs

Simplfy the build script's `main()` function to remove the needless borrow.
Simplfy the build script's `main()` function to use `Option::is_some_and()`
rather than `Option::map_or()`.
…b.rs

Replace `usize::max_value()` with the constant `usize::MAX` in the definition
of `GIT_REBASE_NO_OPERATION`.
@DanielEScherzer
DanielEScherzer force-pushed the clippy-fixes-libgit2-sys branch from df1d114 to 786d2d9 Compare July 19, 2026 20:02

@weihanglo weihanglo left a comment

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.

@weihanglo
weihanglo added this pull request to the merge queue Jul 20, 2026
Merged via the queue into rust-lang:main with commit d4c63f8 Jul 20, 2026
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Waiting on review label Jul 20, 2026
@DanielEScherzer
DanielEScherzer deleted the clippy-fixes-libgit2-sys branch July 20, 2026 01:21
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