improve collapsible if in complex cases with same expressions · rust-lang/rust-clippy#812

2016-03-30T14:34:06.000Z

``` rust if a { if b { f(); } } else { f(); } ``` could be ``` rust if !a || b { f(); } ```

(1 commentaire) (0 réactions) (1 assigné)Rust (1 391 forks)batch import

A-lintC-enhancementgood first issue

Métriques du dépôt

if a {
    if b {
        f();
    }
} else {
    f();
}

could be

if !a || b {
    f();
}

Direction de recherche: Examinez l'implémentation actuelle du lint collapsible if dans rust clippy (fichier clippy lints/src/collapsible if.rs). Comprenez comment il détecte actuellement les instructions if imbriquées simples. L'amélioration nécessiterait d'ajouter une logique pour détecter les cas où la branche else contient la même expression que le corps du if, et de les transformer en une seule condition. Recherchez toute discussion ou tentative existante (l'issue est assignée mais il n'y a pas de PR). Le mainteneur pourrait avoir des avis sur l'approche souhaitée. Concentrez vous sur les modèles AST pour l'exemple donné.
Stack technique: rust
Domaine: tooling
Type d'issue: Fonctionnalité
Difficulté: 3
Temps estimé: 1-2 jours
Statut d'activité: Active
Clarté: Claire
Prérequis: Rust
Accessibilité débutant: 65