llvm/llvm-project

readability check to convert numerical constants to std::numeric_limits

Open

#34.434 geöffnet am 25. Okt. 2017

Auf GitHub ansehen
 (14 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
bugzillacheck-requestclang-tidyconfirmedgood 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 35086
Version unspecified
OS All
CC @JonasToth

Extended Description

It'll be great to have readability check with will convert ~0, -1 for unsigned integer types to std::numeric_limits::max().

Contributor Guide

Research-Richtung
Studieren Sie bestehende clang tidy Lesbarkeitsprüfungen (z. B. readability identifier naming), um die Struktur zu verstehen. Verwenden Sie AST Matcher, um Muster wie das Integer Literal 1 oder ~0, die vorzeichenlosen Typen zugewiesen werden, zu erkennen. Geben Sie dann einen Fix it Hinweis aus, um durch std::numeric limits<type>::max() zu ersetzen. Lesen Sie die clang tidy Dokumentation zum Erstellen von Prüfungen.
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.
1-2 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++clang tidy basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75