llvm/llvm-project

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

Open

#68.213 aberto em 4 de out. de 2023

Ver no GitHub
 (12 comments) (3 reactions) (0 assignees)C++ (10.782 forks)batch import
clang-include-cleanergood first issuehelp wanted

Métricas do repositório

Stars
 (26.378 stars)
Métricas de merge de PR
 (Mesclagem média 1d 2h) (1.000 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
O problema relata que o clang include cleaner sugere incorretamente substituir cabeçalhos padrão como <chrono> pelo cabeçalho interno <bits/chrono.h>. Os diagnósticos indicam que o problema está na lógica do include cleaner para mapear símbolos para cabeçalhos. Investigue o código fonte do clang include cleaner (provavelmente em clang tools extra/clang include cleaner/) para entender como as sugestões são geradas. Os comentários na issue podem fornecer insights sobre o comportamento esperado. Uma correção envolveria verificar se o cabeçalho sugerido é um cabeçalho público e não um cabeçalho interno bits.
Pilha de tecnologia
cpp
Domain
developer experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75