llvm/llvm-project

modernize-use-digit-separator

Open

#35.414 aperta il 23 gen 2018

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

Metriche repository

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

Descrizione
Bugzilla Link 36066
Version unspecified
OS All

Extended Description

Create a check that looks for long integral constants and inserts the digits separator (') appropriately.

Suggested groupings:

  • decimal integral constants, groups of 3 digits, e.g. int x = 1'000;
  • binary integral constants, groups of 4 digits, e.g. int x = 0b0010'0011;
  • octal integral constants, groups of 3 digits, e.g. int x = 0377'777;
  • hexadecimal integral constants, groups of 4 digits, e.g. unsigned long x = 0xffff'0000;
  • floating-point constnats, group into 3 digits on either side of the decimal point, e.g. float x = 3'456.001'25f;

This check requires C++14.

Provision could be made for supplying a mapping of numeric literal type to number of digits to be grouped.

Guida contributor

Direzione di ricerca
Studia i modelli dei controlli clang tidy esistenti, quindi implementa un nuovo controllo che inserisce separatori di cifre nelle costanti intere lunghe secondo i raggruppamenti specificati. Registra il controllo e assicurati che funzioni in modalità C++14.
Tech stack
cpp
Dominio
compilers
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++GitLLVM
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60