llvm/llvm-project

readability check to convert numerical constants to std::numeric_limits

Open

#34.434 aperta il 25 ott 2017

Vedi su GitHub
 (14 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
bugzillacheck-requestclang-tidyconfirmedgood first issue

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione
Bugzilla Link 35086
Version unspecified
OS All
CC @JonasToth

Extended Description

It'll be great to have readability check with will convert ~0, -1 for unsigned integer types to std::numeric_limits::max().

Guida contributor

Direzione di ricerca
Studia i controlli di leggibilità esistenti di clang tidy (es. readability identifier naming) per comprenderne la struttura. Usa i matcher AST per rilevare pattern come il letterale intero 1 o ~0 assegnato a tipi senza segno. Quindi emetti un suggerimento fix it per sostituire con std::numeric limits<type>::max(). Consulta la documentazione di clang tidy per creare controlli.
Tech stack
cpp
Dominio
toolingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++clang tidy basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75