FuelLabs/sway

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

Open

#3 925 ouverte le 30 janv. 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Rust (5 421 forks)batch import
P: lowgood first issue

Métriques du dépôt

Stars
 (61 702 stars)
Métriques de merge PR
 (Merge moyen 5j 4h) (19 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Surveillez la stabilisation de `BTreeMap::drain filter` dans Rust nightly, puis remplacez l'appel `retain` par `drain filter` dans swayfmt/src/comments.rs.
Stack technique
rust
Domaine
tooling
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitRust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70