rust-lang/rust-clippy

Lint request: Prevent lifetimes that are only used for function output

Open

#1874 aperta il 5 lug 2017

Vedi su GitHub
 (9 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
A-lintL-correctnessS-needs-discussionT-middlegood first issue

Metriche repository

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

Descrizione

The following:

fn lifetime<'a>() -> &'a str {
    "Why some people do that"
}

Can be more clearly written as this. Also, using 'static lifetime makes it clear that it was intended, especially in unsafe code where returning unbound lifetime may as well be a mistake.

fn lifetime() -> &'static str {
    "Why some people do that"
}

Guida contributor

Direzione di ricerca
Il problema suggerisce un nuovo lint per rilevare i lifetime utilizzati solo nelle uscite delle funzioni. Inizia esaminando i lint esistenti di clippy relativi ai lifetime (ad es. in clippy lints/src/lifetimes.rs). Comprendi come clippy registra e implementa i lint. Esamina la discussione nei commenti del problema per eventuali vincoli aggiuntivi o alternative. Considera l'output diagnostico e i possibili falsi positivi. Nota che si tratta di una richiesta di funzionalità senza assegnatario, quindi disponibile per il contributo.
Tech stack
rust
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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