rust-lang/rust-clippy

transmute_ptr_to_ref suggestion doesn't with some outer expression types

Open

#1754 aperta il 12 mag 2017

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)Rust (1391 fork)batch import
C-bugT-middlegood first issue

Metriche repository

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

Descrizione

clippy suggests the following replacement

-                mem::transmute::<*mut c_void, &mut Box<FnMut(&Button)>>(data)(&button)
+                &mut *(data as *mut Box<FnMut(&Button)>)(&button)

but we should suggest parens around the &mut *...

bonus points if the parens only appear if necessary

Guida contributor

Direzione di ricerca
Il problema descrive un lint di clippy `transmute ptr to ref` che suggerisce erroneamente di sostituire `mem::transmute::<*mut c void, &mut Box<FnMut(&Button)>>(data)(&button)` con `&mut *(data as *mut Box<FnMut(&Button)>)(&button)` senza parentesi. La correzione comporta la modifica del codice di suggerimento del lint nel repository di clippy (probabilmente in `clippy lints/src/transmute.rs`) per racchiudere `&mut *...` tra parentesi quando appare in un contesto come una chiamata di funzione. L'osservazione è che il suggerimento è sintatticamente ambiguo senza parentesi. Cercare l'implementazione del lint `transmute ptr to ref`, esaminare come viene generato il suggerimento (probabilmente utilizzando `Applicability` e `span lint and sugg`), e aggiungere la logica per rilevare se l'espressione viene utilizzata come chiamato (ad esempio, all'interno di una chiamata di funzione) e quindi aggiungere parentesi attorno al codice suggerito. Testare la modifica con l'esempio fornito e casi simili.
Tech stack
rust
Dominio
developer experiencetooling
Tipo issue
Bug
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.
Abbastanza chiara
Prerequisiti
Rustclippy internal architecture
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75