rust-lang/rust-clippy

Catch transmuting Option<&T> to Option<&mut T>

Open

#372 geöffnet am 8. Okt. 2015

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (1.391 Forks)batch import
C-enhancementE-mediumT-middlegood first issue

Repository-Metriken

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

Beschreibung

Rustc already has a lint (mutable_transmutes) that catches transmutes from &T to &mut T, which can cause undefined behaviour. However, it doesn't catch minor variations of this problem, like Option<&T> -> Option<&mut T>, which are presumably just as dangerous. I've seen this particular example in real code. It would be nice to have a lint that catches this case (and ideally, other instances of the same problem, although this could get quite complex in the fully general case).

Contributor Guide

Research-Richtung
Studieren Sie den vorhandenen `mutable transmutes` Lint im Clippy Quellcode, um zu verstehen, wie er Transmutes von `&T` zu `&mut T` erkennt. Erweitern Sie dann die Logik, um `Option<&T>` zu `Option<&mut T>` und möglicherweise andere Wrappertypen zu behandeln. Ziehen Sie in Betracht, Typauflösung und Pattern Matching auf den Transmute Zieltyp anzuwenden.
Tech Stack
rust
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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.
Klar
Voraussetzungen
RustTransmute semanticsClippy internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60