llvm/llvm-project

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

Open

Aperta il 31 mar 2026

Vedi su GitHub
 (8 commenti) (1 reazione) (1 assegnatario)C++ (26.378 star) (10.782 fork)batch import
check-requestclang-tidygood first issue

Descrizione

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.

Guida contributor

Tech stack
cpp
Dominio
tooling
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 days
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
active
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
C++Clang AST
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
Studiare i pattern dei controlli clang tidy esistenti, specialmente quelli che usano AST matcher, quindi implementare un matcher che rilevi `throw e;` all'interno di un gestore catch dove `e` è l'eccezione catturata, e suggerisca di sostituirlo con `throw;`.