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 建立於 2022年4月21日

在 GitHub 查看
 (12 留言) (0 反應) (0 負責人)C++ (10,782 fork)batch import
clang-tidyenhancementgood first issue

倉庫指標

Star
 (26,378 star)
PR 合併指標
 (平均合併 1天 2小時) (30 天內合併 1,000 個 PR)

描述

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

貢獻者指南