rust-lang/rust-clippy

Suggestion: borrowing a RefCell in funarg position

Open

#4,632 opened on 2019年10月4日

GitHub で見る
 (2 comments) (3 reactions) (0 assignees)Rust (1,391 forks)batch import
A-lintT-middlegood first issue

Repository metrics

Stars
 (10,406 stars)
PR merge metrics
 (平均マージ 19d 22h) (30d で 113 merged PRs)

説明

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?

コントリビューターガイド

調査方針
この機能は、関数引数位置での RefCell 借用を検出する新しい Clippy lint を要求しています。実装するには、借用に関連する既存の Clippy lint（例：borrow deref ref）を調査してください。この lint は、HIR または MIR 内の関数呼び出し式を分析し、引数が RefCell からの借用であるかどうかをチェックする必要があります。また、問題を理解するために、リンクされた Rust issue（https://github.com/rust lang/rust/issues/65086）を確認してください。実装は、なぜそのパターンが問題なのかを説明する明確なメッセージを含む警告を生成する必要があります。
技術スタック
rust
領域
developer experience
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
RustGit
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65