llvm/llvm-project

clang-include-cleaner - replaces chrono with bits/chrono

Open

#68.213 geöffnet am 4. Okt. 2023

Auf GitHub ansehen
 (12 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)C++ (10.782 Forks)batch import
clang-include-cleanergood first issuehelp wanted

Repository-Metriken

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

Beschreibung

Issues found:

a)

+#include <bits/basic_string.h>
+#include <string>

b)

-#include <chrono>
+#include <bits/chrono.h>

Tested version from LLVM 17.0.2

misc-include-cleaner provide more information:

160   - DiagnosticName:  misc-include-cleaner
161     DiagnosticMessage:
162       Message:         'no header providing "std::operator""s" is directly included'
163       FilePath:        'some-file.cpp'
164       FileOffset:      982
165       Replacements:
166         - FilePath:        'some-file.cpp'
167           Offset:          52
168           Length:          0
169           ReplacementText: "#include <bits/basic_string.h>\n"

And:

304   - DiagnosticName:  misc-include-cleaner
305     DiagnosticMessage:
306       Message:         'no header providing "std::chrono::ceil" is directly included'
307       FilePath:        'some-file.cpp'
308       FileOffset:      3133
309       Replacements:
310         - FilePath:        'some-file.cpp'
311           Offset:          43
312           Length:          0
313           ReplacementText: "#include <bits/chrono.h>\n"

When <chrono> is already included.

Contributor Guide

Research-Richtung
Das Problem meldet, dass clang include cleaner fälschlicherweise vorschlägt, Standard Header wie <chrono> durch interne Header wie <bits/chrono.h> zu ersetzen. Die Diagnoseinformationen deuten darauf hin, dass das Problem in der Logik des Include Cleaners zur Zuordnung von Symbolen zu Headern liegt. Untersuchen Sie den Quellcode von clang include cleaner (wahrscheinlich unter clang tools extra/clang include cleaner/), um zu verstehen, wie die Vorschläge generiert werden. Kommentare im Issue können Einblicke in das erwartete Verhalten geben. Eine Korrektur würde beinhalten, zu prüfen, ob der vorgeschlagene Header ein öffentlicher Header und kein interner Bits Header ist.
Tech Stack
cpp
Domain
developer experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75