Repository navigation
Address most compiler/clippy warnings - #865
Merged
Merged
Conversation
Contributor
|
This would be nice to have merged, reduces warnings fatigue and makes focusing on actual (new) errors easier |
azat
added a commit
to azat-rust/cursive
that referenced
this pull request
Jun 11, 2026
* origin/main: Add horizontal scroll gesture support (gyscos#872) Address most compiler/clippy warnings (gyscos#865) Fix focus_view for Dialog and LinearLayout Update third-party views in README (gyscos#864) Remove mention of inexistent example Update rand dev-dependency to 0.10 Update find-crate requirement from 0.6.3 to 0.7.1 (gyscos#858) Update toml requirement from 0.9 to 1.0 (gyscos#860)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I've been working on #857, and have noticed when trying to address any warnings in my code, that they're getting a bit lost in other warnings. This PR just addresses the warnings that have obvious solutions, which ended up being all of them except for one dead code warning.