rust-lang/rust-clippy

Lint unwrap in drop impls

Open

#1.523 geöffnet am 8. Feb. 2017

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.391 Forks)batch import
A-lintE-hardL-correctnessT-middlegood first issue

Repository-Metriken

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

Beschreibung

Would it make sense to deny panic-able fns in Drop::drop implementations? It recently came up in https://github.com/diesel-rs/diesel/pull/641#discussion_r100066589.

Contributor Guide

Research-Richtung
Studieren Sie vorhandene Clippy Lints, die auf Panics prüfen (z.B. `panic` Lint), und verstehen Sie das Lint Registrierungssystem. Implementieren Sie dann einen neuen Lint, der unwrap Aufrufe in `Drop` Implementierungen erkennt, indem Sie die Funktionskörper von `drop` Methoden durchlaufen.
Tech Stack
rust
Domain
developer experience
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-2 Tage
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
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60