improve collapsible if in complex cases with same expressions

Métriques du dépôt

Stars

10 406 étoiles (10 406 étoiles)

Métriques de merge PR

Merge moyen 19j 22h (Merge moyen 19j 22h)113 PRs mergées en 30 j (113 PRs mergées en 30 j)

Description

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

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

Guide contributeur

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é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.

65