rust-lang/rust-clippy

Suggestion: borrowing a RefCell in funarg position

Open

#4 632 ouverte le 4 oct. 2019

Voir sur GitHub
 (2 commentaires) (3 réactions) (0 assignés)Rust (1 391 forks)batch import
A-lintT-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

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?

Guide contributeur

Direction de recherche
Cette fonctionnalité demande un nouveau lint Clippy pour détecter les emprunts de RefCell dans les positions d'arguments de fonction. Pour l'implémenter, étudiez les lints Clippy existants liés aux emprunts (par exemple, borrow deref ref). Le lint devra analyser les expressions d'appel de fonction dans le HIR ou le MIR, en vérifiant si un argument est un emprunt depuis un RefCell. Consultez également l'issue Rust liée (https://github.com/rust lang/rust/issues/65086) pour comprendre le problème. L'implémentation doit produire un avertissement avec un message clair expliquant pourquoi le motif est problématique.
Stack technique
rust
Domaine
developer experience
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65