llvm/llvm-project

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

Open

#36 323 ouverte le 2 avr. 2018

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)C++ (10 782 forks)batch import
bugzillaclang-tidyenhancementgood first issue

Métriques du dépôt

Stars
 (26 378 stars)
Métriques de merge PR
 (Merge moyen 1j 2h) (1 000 PRs mergées en 30 j)

Description
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.

Guide contributeur

Direction de recherche
Le problème demande d'ajouter des options à la vérification readability implicit bool conversion dans clang tidy. Commencez par examiner l'implémentation existante de la vérification dans clang tools extra/clang tidy/readability/ImplicitBoolConversionCheck.cpp. Recherchez la gestion actuelle des options et ajoutez deux nouvelles options booléennes : AllowConversionsToBool et AllowConversionsFromBool. Modifiez la logique de vérification pour supprimer les avertissements en fonction de ces options. Mettez à jour la documentation de la vérification et ajoutez des cas de test dans clang tools extra/test/clang tidy/checkers/readability implicit bool conversion.cpp. Consultez les commentaires du problème pour un contexte supplémentaire.
Stack technique
cpp
Domaine
toolingdeveloper experience
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++GitLLVM build system
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65