diff options
| author | Victor Chernyakin <chernyakin.victor.j@outlook.com> | 2025-07-08 10:48:51 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-07-08 20:48:51 +0300 |
| commit | 1e3f6a6c4fd8dafb208216ffcabe31fa978ca502 (patch) | |
| tree | cfaff3a2614f0fcd557928c50dc9fd0b752ca152 /clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp | |
| parent | e976eaf3037a1c0fe68432ae60a503bf144a1c30 (diff) | |
| download | llvm-1e3f6a6c4fd8dafb208216ffcabe31fa978ca502.zip llvm-1e3f6a6c4fd8dafb208216ffcabe31fa978ca502.tar.gz llvm-1e3f6a6c4fd8dafb208216ffcabe31fa978ca502.tar.bz2 | |
[clang-tidy][NFC] Prefer `constexpr llvm::StringLiteral` over `const char *` (#147301)
Some of these are even global mutable state — probably not what was
intended!
```cpp
static const char *AnalyzerCheckNamePrefix = "clang-analyzer-";
```
Diffstat (limited to 'clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp index 30fcba3..936a906 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp @@ -119,7 +119,8 @@ void UseUsingCheck::check(const MatchFinder::MatchResult &Result) { if (StartLoc.isMacroID() && IgnoreMacros) return; - static const char *UseUsingWarning = "use 'using' instead of 'typedef'"; + static constexpr llvm::StringLiteral UseUsingWarning = + "use 'using' instead of 'typedef'"; // Warn at StartLoc but do not fix if there is macro or array. if (MatchedDecl->getUnderlyingType()->isArrayType() || StartLoc.isMacroID()) { |