llvm/llvm-project

modernize-use-digit-separator

Aperta

#35.414 aperta il 23 gen 2018

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

Metriche repository

Star
 (26.378 stelle)
Metriche merge PR
 (Metriche PR in attesa)

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