FuelLabs/sway

refactor(fmt): use `drain_filter` during comment formatting when stabilized

Open

#3,925 opened on 2023年1月30日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)Rust (5,421 forks)batch import
P: lowgood first issue

Repository metrics

Stars
 (61,702 stars)
PR merge metrics
 (平均マージ 22h 6m) (30d で 16 merged PRs)

説明

BTreeMap does not have an equivalent for Range that also removes the yielded elements, however it does have a > drain_filter method.

A more efficient (performance and code-wise) alternative to this function using drain_filter might look like:

formatter.comment_map.drain_filter(|bs, _v| bs.start >= start && bs.end <= end)

Originally posted by @mitchmindtree in https://github.com/FuelLabs/sway/pull/3854#discussion_r1083691334

Currently, drain_filter is a a nightly-only API - I opted to use retain instead to filter out already formatted comments. We can perhaps consider doing this once drain_filter is stabilized.

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

調査方針
Rust ナイトリービルドで `BTreeMap::drain filter` の安定化を監視し、swayfmt/src/comments.rs で `retain` 呼び出しを `drain filter` に置き換えます。
技術スタック
rust
領域
tooling
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
古い
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GitRust
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70