rust-lang/rust-clippy

Suggestion: borrowing a RefCell in funarg position

Open

#4632 aperta il 4 ott 2019

Vedi su GitHub
 (2 commenti) (3 reazioni) (0 assegnatari)Rust (1391 fork)batch import
A-lintT-middlegood first issue

Metriche repository

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

Descrizione

I recently came across this fun correctness bug in my code: https://github.com/rust-lang/rust/issues/65086.

I’m not sure how I could have known this without asking a compiler developer. Can we add a lint prohibiting RefCell borrows in funarg position since they prevent mutable borrows further down the stack?

Guida contributor

Direzione di ricerca
Questa funzionalità richiede un nuovo lint di Clippy per rilevare i prestiti di RefCell nelle posizioni degli argomenti di funzione. Per implementarlo, studia i lint esistenti di Clippy relativi ai prestiti (ad esempio, borrow deref ref). Il lint dovrà analizzare le espressioni di chiamata di funzione nell'HIR o nel MIR, verificando se qualche argomento è un prestito da un RefCell. Inoltre, rivedi l'issue di Rust collegata (https://github.com/rust lang/rust/issues/65086) per comprendere il problema. L'implementazione dovrebbe produrre un avviso con un messaggio chiaro che spiega perché il pattern è problematico.
Tech stack
rust
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65