rust-lang/rust-clippy

Extract commonly used visitors to utils

Open

#4,413 opened on 2019年8月19日

GitHub で見る
 (3 comments) (1 reaction) (0 assignees)Rust (10,406 stars) (1,391 forks)batch import
C-enhancementgood first issue

説明

Both #3427 and #4386 implement a Visitor that walks an expression and searches for a return. This should be de-duplicated.

Maybe there are other Visitors in the Clippy code base, that also do similar things and can be extracted to utils.

Originally posted by @mikerite in https://github.com/rust-lang/rust-clippy/pull/4386#issuecomment-522407926

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

技術スタック
rust
領域
tooling
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
RustClippy internalsVisitor pattern
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
35
調査方針
Examine the linked PRs #3427 and #4386 to understand the duplicated Visitor implementations. Search the Clippy source code (likely in `clippy lints/src/`) for similar Visitors that traverse expressions and check for returns. Extract the common logic into a utility module, perhaps `clippy utils/src/visitors.rs`. Refer to issue comments for further details on which visitors to consolidate.