llvm/llvm-project

[clang-tidy] Add `readability-use-rethrow` check

Open

#189 672 ouverte le 31 mars 2026

Voir sur GitHub
 (8 commentaires) (1 réaction) (1 assigné)C++ (10 782 forks)batch import
check-requestclang-tidygood first issue

Métriques du dépôt

Stars
 (26 378 stars)
Métriques de merge PR
 (Merge moyen 1j 2h) (1 000 PRs mergées en 30 j)

Description

It would be useful to have a clang-tidy check that detects cases where code is clearly trying to rethrow the currently handled exception, but spells it as throw e; instead of throw;.

For example, code like this is easy to write:

try {
  f();
} catch (const std::exception &e) {
  log(e.what());
  throw e;
}

but if the intent is to rethrow the active exception, the clearer spelling is:

try {
  f();
} catch (const std::exception &e) {
  log(e.what());
  throw;
}

The latter expresses the intent directly and avoids the impression that the code is deliberately throwing the caught object as a new exception expression.

Guide contributeur

Direction de recherche
Etudiez les motifs des checks clang tidy existants, en particulier ceux utilisant des AST matchers, puis implementez un matcher qui detecte `throw e;` dans un gestionnaire catch ou `e` est l'exception attrapee, et suggere de le remplacer par `throw;`.
Stack technique
cpp
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.
3-5 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
C++Clang AST
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70