llvm/llvm-project

clang-tidy check readability-container-data-pointer fix isn't configurable to use std::string::c_str() instead of data()

Open

#55.026 aberto em 21 de abr. de 2022

Ver no GitHub
 (12 comments) (0 reactions) (0 assignees)C++ (10.782 forks)batch import
clang-tidyenhancementgood first issue

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

This is an "enhancement" request. I used clang-tidy 14.0.0.

Use C++ code

#include <string>

int main()
{
    std::string st1;
    char *cp1 = &st1[0];
    return 0;
}

The clang-tidy fix creates st1.data() -- ideally the fix would create std.c_str() instead or have a fix configuration option for that.

However, usage of c_str() would only work if the "const char* data()" overload were used instead of "char * data()".

Guia do colaborador

Direção de pesquisa
Examine a implementação da verificação readability container data pointer do clang tidy no código fonte do LLVM. Adicione uma opção de configuração (por exemplo, 'UseCStr') que, quando ativada, substitui data() por c str() para std::string, mas apenas quando a sobrecarga const estiver disponível. Atualize os testes e a documentação.
Pilha de tecnologia
cpp
Domain
developer experiencetooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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++LLVM/clang tidy internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50