llvm/llvm-project

Add option to readability-implicit-bool-conversion check to allow implicit conversion *to bool*

Open

#36.323 aperta il 2 apr 2018

Vedi su GitHub
 (4 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
bugzillaclang-tidyenhancementgood first issue

Metriche repository

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

Descrizione
Bugzilla Link 36975
Version unspecified
OS All
Reporter LLVM Bugzilla Contributor

Extended Description

From the readability-implicit-bool-conversion check, I'd like to take advantage of these parts:

  • boolean expression/literal to integer,
  • boolean expression/literal to floating.

But not these:

  • integer expression/literal to boolean,
  • floating expression/literal to boolean,
  • pointer/pointer to member/nullptr/NULL to boolean,

Seems like an AllowConversionsToBool option would take care of this. Maybe for completeness/symmetry also a AllowConversionsFromBool option. Both defaulting to 0.

Guida contributor

Direzione di ricerca
La richiesta del problema è di aggiungere opzioni al controllo readability implicit bool conversion in clang tidy. Inizia esaminando l'implementazione esistente del controllo in clang tools extra/clang tidy/readability/ImplicitBoolConversionCheck.cpp. Cerca la gestione delle opzioni attuali e aggiungi due nuove opzioni booleane: AllowConversionsToBool e AllowConversionsFromBool. Modifica la logica del controllo per sopprimere gli avvisi in base a queste opzioni. Aggiorna la documentazione del controllo e aggiungi casi di test in clang tools extra/test/clang tidy/checkers/readability implicit bool conversion.cpp. Rivedi i commenti del problema per ulteriore contesto.
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.
3-5 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 build system
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65