llvm/llvm-project

Readability check for position of constant operand in binary operators

Open

#31.860 geöffnet am 3. Apr. 2017

Auf GitHub ansehen
 (13 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
bugzillacheck-requestclang-tidygood 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 32513
Version unspecified
OS All

Extended Description

Will be good idea to have readability check for position of constant operand in binary operators.

For example:

if (nullptr == Pointer)

should be fixed as

if (Pointer == nullptr)

or

if (0 < Value)

to

if (Value > 0)

Contributor Guide

Research-Richtung
Implementieren Sie einen neuen clang tidy Check im Lesbarkeitsmodul, um konstante Operanden auf der linken Seite von binären Operatoren zu erkennen und zu korrigieren. Siehe vorhandene Checks wie 'readability redundant *' für Muster.
Tech Stack
cpp
Domain
compilersdevtools
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.
1-3 Stunden
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.
Meist klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55