rust-lang/rust-clippy

transmute_ptr_to_ref suggestion doesn't with some outer expression types

Open

#1 754 ouverte le 12 mai 2017

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)Rust (1 391 forks)batch import
C-bugT-middlegood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Le problème décrit un lint clippy `transmute ptr to ref` qui suggère incorrectement de remplacer `mem::transmute::<*mut c void, &mut Box<FnMut(&Button)>>(data)(&button)` par `&mut *(data as *mut Box<FnMut(&Button)>)(&button)` sans parenthèses. La correction implique de modifier le code de suggestion du lint dans le dépôt clippy (probablement dans `clippy lints/src/transmute.rs`) pour entourer `&mut *...` de parenthèses lorsqu'il apparaît dans un contexte comme un appel de fonction. L'observation est que la suggestion est syntaxiquement ambiguë sans parenthèses. Cherchez l'implémentation du lint `transmute ptr to ref`, examinez comment la suggestion est générée (probablement en utilisant `Applicability` et `span lint and sugg`), et ajoutez une logique pour détecter si l'expression est utilisée comme appelé (par exemple, à l'intérieur d'un appel de fonction), puis ajoutez des parenthèses autour du code suggéré. Testez le changement avec l'exemple fourni et des cas similaires.
Stack technique
rust
Domaine
developer experiencetooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rustclippy internal architecture
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75