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 aperta il 21 apr 2022

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)C++ (10.782 fork)batch import
clang-tidyenhancementgood first issue

Metriche repository

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

Descrizione

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()".

Guida contributor

Direzione di ricerca
Esamina l'implementazione del controllo readability container data pointer di clang tidy nel sorgente LLVM. Aggiungi un'opzione di configurazione (ad esempio 'UseCStr') che, quando abilitata, sostituisce data() con c str() per std::string, ma solo quando il sovraccarico const è disponibile. Aggiorna i test e la documentazione.
Tech stack
cpp
Dominio
developer experiencetooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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++LLVM/clang tidy internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50