llvm/llvm-project

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

Open

#68.213 aperta il 4 ott 2023

Vedi su GitHub
 (12 commenti) (3 reazioni) (0 assegnatari)C++ (10.782 fork)batch import
clang-include-cleanergood first issuehelp wanted

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema segnala che clang include cleaner suggerisce erroneamente di sostituire intestazioni standard come <chrono> con intestazioni interne come <bits/chrono.h>. Le diagnostiche indicano che il problema risiede nella logica di mappatura dei simboli alle intestazioni di include cleaner. Esamina il codice sorgente di clang include cleaner (probabilmente in clang tools extra/clang include cleaner/) per capire come vengono generati i suggerimenti. I commenti nell'issue potrebbero fornire spunti sul comportamento atteso. Una correzione richiederebbe di verificare che l'intestazione suggerita sia un'intestazione pubblica e non un'intestazione interna bits.
Tech stack
cpp
Dominio
developer experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75