FuelLabs/sway

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

Open

#3925 aperta il 30 gen 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Rust (5421 fork)batch import
P: lowgood first issue

Metriche repository

Star
 (61.702 star)
Metriche merge PR
 (Merge medio 22h 6m) (16 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Monitora la stabilizzazione di `BTreeMap::drain filter` in Rust nightly, quindi sostituisci la chiamata a `retain` con `drain filter` in swayfmt/src/comments.rs.
Tech stack
rust
Dominio
tooling
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitRust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70