Skip to content

rm: improve the code a tiny bit#5755

Merged
cakebaker merged 1 commit intouutils:mainfrom
sylvestre:syntax
Dec 31, 2023
Merged

rm: improve the code a tiny bit#5755
cakebaker merged 1 commit intouutils:mainfrom
sylvestre:syntax

Conversation

@sylvestre
Copy link
Copy Markdown
Contributor

No description provided.

@cakebaker cakebaker merged commit ed11132 into uutils:main Dec 31, 2023
@sylvestre
Copy link
Copy Markdown
Contributor Author

@samueltardieu maybe you might be interested by this simple clippy potential rule :)

@samueltardieu
Copy link
Copy Markdown
Contributor

samueltardieu commented Dec 31, 2023

maybe you might be interested by this simple clippy potential rule

Clippy assumes that _e is there for informative reasons and will not propose to remove it (it should have suggested if matches!(e, Err(_e)) though. If it had been written as if let Err(_) = e then .is_err() would have been suggested.

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