rust-lang/rust-clippy

Lint request: Prevent lifetimes that are only used for function output

Open

#1 874 ouverte le 5 juil. 2017

Voir sur GitHub
 (9 commentaires) (0 réactions) (0 assignés)Rust (1 391 forks)batch import
A-lintL-correctnessS-needs-discussionT-middlegood 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

The following:

fn lifetime<'a>() -> &'a str {
    "Why some people do that"
}

Can be more clearly written as this. Also, using 'static lifetime makes it clear that it was intended, especially in unsafe code where returning unbound lifetime may as well be a mistake.

fn lifetime() -> &'static str {
    "Why some people do that"
}

Guide contributeur

Direction de recherche
Le problème suggère un nouveau lint pour détecter les durées de vie qui ne sont utilisées que dans les sorties de fonction. Commencez par examiner les lints existants de clippy liés aux durées de vie (par exemple dans clippy lints/src/lifetimes.rs). Comprenez comment clippy enregistre et implémente les lints. Consultez la discussion dans les commentaires du problème pour toute contrainte ou alternative supplémentaire. Prenez en compte la sortie de diagnostic et les faux positifs potentiels. Notez qu'il s'agit d'une demande de fonctionnalité sans responsable attribué, elle est donc disponible pour contribution.
Stack technique
rust
Domaine
backend
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
70