Project

General

Profile

Actions

Optimization #4591

closed

Fix Rust clippy lints

Added by Juliana Fajardini Reichow over 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:
Rust

Description

Umbrella ticket for issues related to fixing Rust Clippy lints.
The goal is to clean Rust code from lint-related warnings. Each ticket covers a specific warning. The complete list is:
- https://rust-lang.github.io/rust-clippy/master/index.html#single_match
- https://rust-lang.github.io/rust-clippy/master/index.html#cmp_null
- https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
- https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
- https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
- https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
- https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
- https://rust-lang.github.io/rust-clippy/master/index.html#module_inception
- https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
- https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names
- https://rust-lang.github.io/rust-clippy/master/index.html#if_same_then_else
- https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
- https://rust-lang.github.io/rust-clippy/master/index.html#match_single_binding
- https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
- https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_lifetimes
- https://rust-lang.github.io/rust-clippy/master/index.html#mixed_case_hex_literals
- https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity
- https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool
- https://rust-lang.github.io/rust-clippy/master/index.html#never_loop
- https://rust-lang.github.io/rust-clippy/master/index.html#large_enum_variant
- https://rust-lang.github.io/rust-clippy/master/index.html#for_loops_over_fallibles
- https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop
- https://rust-lang.github.io/rust-clippy/master/index.html#branches_sharing_code
- https://rust-lang.github.io/rust-clippy/master/index.html#while_let_loop
- https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching
- https://rust-lang.github.io/rust-clippy/master/index.html#inherent_to_string
- https://rust-lang.github.io/rust-clippy/master/index.html#field_reassign_with_default
- https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_match

Clippy is "a collection of lints to catch common mistakes and improve Rust code".

Running

cargo clippy --all-features

Will show all existing warnings, which must be fixed manually.

For more information and an installation guide: https://github.com/rust-lang/rust-clippy#as-a-cargo-subcommand-cargo-clippy


Subtasks 26 (0 open26 closed)

Optimization #4592: Fix warning about "for loop over fallibles"ClosedJason IshActions
Optimization #4593: Fix warning about "mixed case hex literals"ClosedAlice da Silva AkakiActions
Optimization #4594: Fix warning about "this loop never actually loops"ClosedJason IshActions
Optimization #4595: Fix warning about "comparing with null"ClosedSam MohammadActions
Optimization #4596: Fix warning about "single match"ClosedJason IshActions
Optimization #4597: Fix warning about "enum's name"ClosedHaleema KhanActions
Optimization #4598: Fix warning about "needless_range_loop"ClosedJason IshActions
Optimization #4599: Fix warning about "ptr_arg"ClosedJason IshActions
Optimization #4600: Fix warning about "needless lifetimes"ClosedJason IshActions
Optimization #4601: Fix warning about "while let loop"ClosedJason IshActions
Optimization #4602: Fix warning about "new without default"ClosedJason IshActions
Optimization #4603: Fix warning about "type complexity"ClosedJason IshActions
Optimization #4604: Fix warning about "branches sharing code"ClosedJason IshActions
Optimization #4605: Fix warning about "unnecessary nested match"ClosedModupe FalodunActions
Optimization #4606: Fix warning about "match ref pats"ClosedJason IshActions
Optimization #4607: Fix warning about "match like matches macro"RejectedActions
Optimization #4608: Fix warning about "redundant pattern matching"ClosedJason IshActions
Optimization #4609: Fix warning about "if same then else"ClosedGabriel Lima LuzActions
Optimization #4610: Fix warning about "explicit counter loop"ClosedJason IshActions
Optimization #4611: Fix warning about "extra unused lifetimes"ClosedHaleema KhanActions
Optimization #4612: Fix warning about "nonminimal bool"ClosedJason IshActions
Optimization #4613: Fix warning about "large enum variant"ClosedJason IshActions
Optimization #4614: Fix warning about "field reassign with default"ClosedJason IshActions
Optimization #4616: Fix warning about "match single binding"ClosedSam MohammadActions
Optimization #4618: Fix warning about "inherent to string"ClosedModupe FalodunActions
Optimization #5577: Fix warning about "comparing with null" in debug codeClosedJason IshActions
Actions

Also available in: Atom PDF