rust-lang/rust-clippy

FR: redundant_field_names should distinguish between all and some fields matching

Open

#3422 aperta il 10 nov 2018

Vedi su GitHub
 (1 commento) (4 reazioni) (1 assegnatario)Rust (1391 fork)batch import
C-enhancementL-stylegood first issue

Metriche repository

Star
 (10.406 star)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

redundant_field_names fires on any field being a match. This impairs readability in cases like this:

let thingy = Thingy {
   calc: big + calculation,
   redundant: redundant,
   other: stuff.into(),
};

As you can see, if you apply the suggestion, you end up with a weird line that is different from the rest. I suggest that the lint be split such that it distinguishes between "all fields are redundant" and "some fields are redundant".

Guida contributor

Direzione di ricerca
Esamina l'attuale implementazione del lint `redundant field names` in Clippy e modificalo per rilevare se tutti i campi o solo alcuni sono ridondanti, quindi adatta il suggerimento del lint di conseguenza.
Tech stack
rust
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70