rust-lang/rust-clippy

Suggestion: borrowing a RefCell in funarg position

Open

#4.632 geöffnet am 4. Okt. 2019

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

Repository-Metriken

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

Beschreibung

I recently came across this fun correctness bug in my code: https://github.com/rust-lang/rust/issues/65086.

I’m not sure how I could have known this without asking a compiler developer. Can we add a lint prohibiting RefCell borrows in funarg position since they prevent mutable borrows further down the stack?

Contributor Guide

Research-Richtung
Diese Funktion fordert einen neuen Clippy Lint an, um RefCell Borrows in Funktionsargumentpositionen zu erkennen. Zur Implementierung sollten bestehende Clippy Lints im Zusammenhang mit Borrows (z. B. borrow deref ref) untersucht werden. Der Lint muss Funktionsaufrufausdrücke im HIR oder MIR analysieren und prüfen, ob ein Argument ein Borrow von einem RefCell ist. Überprüfen Sie außerdem das verlinkte Rust Issue (https://github.com/rust lang/rust/issues/65086), um das Problem zu verstehen. Die Implementierung sollte eine Warnung mit einer klaren Nachricht ausgeben, die erklärt, warum das Muster problematisch ist.
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
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65