rust-lang/rust-clippy

transmute_ptr_to_ref suggestion doesn't with some outer expression types

Open

#1.754 geöffnet am 12. Mai 2017

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)Rust (1.391 Forks)batch import
C-bugT-middlegood first issue

Repository-Metriken

Stars
 (10.406 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 19T 22h) (113 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem beschreibt einen clippy lint `transmute ptr to ref`, der fälschlicherweise vorschlägt, `mem::transmute::<*mut c void, &mut Box<FnMut(&Button)>>(data)(&button)` durch `&mut *(data as *mut Box<FnMut(&Button)>)(&button)` ohne Klammern zu ersetzen. Die Korrektur beinhaltet die Änderung des Vorschlagscodes des Lints im clippy Repository (wahrscheinlich in `clippy lints/src/transmute.rs`), um `&mut *...` in Klammern zu setzen, wenn es in einem Kontext wie einem Funktionsaufruf erscheint. Die Beobachtung ist, dass der Vorschlag ohne Klammern syntaktisch mehrdeutig ist. Suchen Sie die Implementierung des `transmute ptr to ref` Lints, untersuchen Sie, wie der Vorschlag generiert wird (wahrscheinlich mit `Applicability` und `span lint and sugg`), und fügen Sie Logik hinzu, um zu erkennen, ob der Ausdruck als Aufgerufener (z. B. innerhalb eines Funktionsaufrufs) verwendet wird, und setzen Sie dann Klammern um den vorgeschlagenen Code. Testen Sie die Änderung mit dem bereitgestellten Beispiel und ähnlichen Fällen.
Tech Stack
rust
Domain
developer experiencetooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Rustclippy internal architecture
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75