llvm/llvm-project

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

Open

#36.323 geöffnet am 2. Apr. 2018

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
bugzillaclang-tidyenhancementgood first issue

Repository-Metriken

Stars
 (26.378 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 2h) (1.000 gemergte PRs in 30 T)

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

Contributor Guide

Research-Richtung
Das Issue fordert die Hinzufügung von Optionen zur Prüfung readability implicit bool conversion in clang tidy. Beginnen Sie mit der Untersuchung der bestehenden Prüfungsimplementierung in clang tools extra/clang tidy/readability/ImplicitBoolConversionCheck.cpp. Suchen Sie nach der aktuellen Optionenbehandlung und fügen Sie zwei neue boolesche Optionen hinzu: AllowConversionsToBool und AllowConversionsFromBool. Passen Sie die Prüfungslogik an, um Warnungen basierend auf diesen Optionen zu unterdrücken. Aktualisieren Sie die Dokumentation der Prüfung und fügen Sie Testfälle in clang tools extra/test/clang tidy/checkers/readability implicit bool conversion.cpp hinzu. Überprüfen Sie die Kommentare im Issue für weiteren Kontext.
Tech Stack
cpp
Domain
toolingdeveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++GitLLVM build system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65