rust-lang/rust-clippy

lint long chains of into/float additions/multiplications

Open

#1 243 ouverte le 25 sept. 2016

Voir sur GitHub
 (13 commentaires) (0 réactions) (0 assignés)Rust (1 391 forks)batch import
A-lintL-perfT-ASTgood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

In generall, (a+b+c)+(d+e+f) is faster to compute than a+b+c+d+e+f, because the former reduces data dependencies, allowing the CPU to run additions in parallel. For float arithmetic, the tree-like addition may improve numerical stability. Suggest inserting parenthesis.

Guide contributeur

Direction de recherche
Étudiez les lints existants de clippy pour l'arithmétique (par exemple `clippy::precedence`) et comprenez comment ajouter un nouveau lint. Ensuite, implémentez un lint qui détecte les longues chaînes d'additions/multiplications et suggère d'insérer des parenthèses.
Stack technique
rust
Domaine
devtools
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50