rust-lang/rust-clippy

from_iter_instead_of_collect / needless_collect interaction questionable

Geschlossen

#6.533 geöffnet am 01.01.2021

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.391 Forks)batch import
C-enhancementI-false-negativeL-nurserygood first issue

Repository-Metriken

Stars
 (10.406 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 19T 22h) (113 gemergte PRs in 30 T)

Beschreibung

for _i in Vec::from_iter([1, 2, 3].iter()) {}

clippy will suggest to use .collect() instead of from_iter() here but blindly replacing will not work because .collect() will need some kind of type hint.

Them again

for _i in [1, 2, 3].iter().collect::<Vec<&i32>>() {}

does not trigger clippy::needless_collect (although it should!; neither does the first example) because we can iterate over

for _i in [1, 2, 3].iter() {}

directly.

Contributor Guide

Research-Richtung
Untersuche die Lints `from iter instead of collect` und `needless collect` im Quellcode von Clippy, wahrscheinlich in Dateien wie `clippy lints/src/from iter instead of collect.rs` und `clippy lints/src/needless collect.rs`. Stelle fest, wie sie interagieren und warum der Vorschlag, `collect()` zu verwenden, ohne Typ Hinweise fehlschlägt. Erwäge, den Lint anzupassen, um entweder einen geeigneten Typ Hinweis vorzuschlagen oder den Vorschlag zu unterdrücken, wenn er einen Fehler verursachen würde. Prüfe auch, ob `needless collect` für den `from iter` Fall ausgelöst werden sollte.
Tech Stack
rust
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Basic understanding of Rust lintsFamiliarity with Clippy's lint structure
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40